Merge lp:~feng-kylin/unity/changeTopLabel into lp:unity
Proposed by
handsome_feng
Status: | Rejected |
---|---|
Rejected by: | Stephen M. Webb |
Proposed branch: | lp:~feng-kylin/unity/changeTopLabel |
Merge into: | lp:unity |
Diff against target: |
74 lines (+27/-3) (has conflicts) 3 files modified
debian/changelog (+9/-0) panel/PanelMenuView.cpp (+17/-2) panel/PanelMenuView.h (+1/-1) Text conflict in debian/changelog |
To merge this branch: | bzr merge lp:~feng-kylin/unity/changeTopLabel |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Sebastien Bacher | Needs Information | ||
Anthony Wong (community) | Needs Fixing | ||
Review via email: mp+223190@code.launchpad.net |
Description of the change
The desktop name "Ubuntu Desktop" can not be notified,it is not convenient for us to change it to our Special symbol,so i remove the ”const" attribute of desktop_name_ in PanelManuView.h ,and add a if statement in PanelManuView.cpp to judge witch desktop_name to show .thank you.
To post a comment you must log in.
Unmerged revisions
- 3828. By handsome_feng
-
Add support for getting the distro name from /etc/os-release
- 3827. By handsome_feng
-
change the 'Ubuntu Desktop' to _('Ubuntu Desktop')
- 3826. By handsome_feng
-
change the label on the panel menu
- 3825. By handsome_feng
-
add a if statement,just for practice.
- 3824. By handsome_feng
-
add a if statement (a novice,make this for practice)
The string "Ubuntu Desktop" was localized but in your change it is no longer the case. You should use _("Ubuntu Desktop") to take the localized string. The same applies to the "Ubuntu Kylin Desktop" string.