unity/plugins/unityshell.xml.in needs some love :)
Bug #1067113 reported by
MC Return
This bug affects 2 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Unity |
Fix Released
|
Undecided
|
MC Return | ||
| unity (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
Bug Description
Currently this file and the resulting CCSM plug-in unityshell is very disorganized.
Settings have been added by several contributors over time resulting in one big chaos.
The main problems are:
1. Wrong indentation makes the .xml.in file hard to read
2. Settings are mixed up and tabs like "Behaviour" and "Experimental" make it hard to find specific settings in the plug-in
3. Tooltips do not use the same style and also contain errors
4. There is no real structuring, nor are options, which belong together, next to each other
5. There is no consequent style used inside the .xml.in file, which also increases difficulty for humans reading it
Related branches
lp:~mc-return/unity/unity.merge-unityshell-compiz-ccsm-plug-in-refactoring
- Brandon Schaefer (community): Approve
- PS Jenkins bot: Pending (continuous-integration) requested
-
Diff: 745 lines (+344/-294)1 file modifiedplugins/unityshell/unityshell.xml.in (+344/-294)
Revision history for this message
| MC Return (mc-return) wrote | #1 |
Revision history for this message
| MC Return (mc-return) wrote | #2 |
Revision history for this message
| MC Return (mc-return) wrote | #3 |
Revision history for this message
| MC Return (mc-return) wrote | #4 |
| Changed in unity: | |
| assignee: | nobody → MC Return (mc-return) |
| Changed in unity: | |
| status: | New → Confirmed |
Revision history for this message
| MC Return (mc-return) wrote | #5 |
Revision history for this message
| MC Return (mc-return) wrote | #6 |
Revision history for this message
| MC Return (mc-return) wrote | #7 |
Revision history for this message
| MC Return (mc-return) wrote | #8 |
| Changed in unity: | |
| status: | Confirmed → Fix Committed |
| Changed in unity: | |
| status: | Fix Committed → Fix Released |
| Changed in unity (Ubuntu): | |
| status: | New → Fix Released |
To post a comment you must log in.
The linked branch fixes all of those problems.