Bug 299317

Summary: JJ: Toolbar icon tooltip texts are inconsistent regarding capitalization and style
Product: [Applications] kdevelop Reporter: Alexander Shaduri <ashaduri>
Component: generalAssignee: kdevelop-bugs-null
Status: RESOLVED FIXED    
Severity: minor Keywords: junior-jobs
Priority: NOR    
Version: git master   
Target Milestone: 4.3.0   
Platform: openSUSE   
OS: Linux   
Latest Commit: Version Fixed In: 4.6.0

Description Alexander Shaduri 2012-05-03 17:53:01 UTC
Hello,

I noticed there are inconsistencies in tooltip texts of KDevelop's toolbar buttons. This makes the program look somewhat unprofessional (which is a shame, considering its features). Here are the ones I noticed:

Main toolbar:
"Execute current Launch" (variable capitalization).
"Debug current Launch" (variable capitalization).

Debugger toolbar:
"Continues the application execution" (should be "Continue application execution").
"Steps out of the current function" (should be "Step out of the current function").

Note that there is also an inconsistency regarding the style of capitalization (Sentence vs Title, see http://techbase.kde.org/Projects/Usability/HIG/Capitalization ). If I understand it correctly, the tooltips should use the sentence-style capitalization ("Pop-up hint text"). (GNOME HIG also dictates sentence-style for tooltips).
Either way, KDevelop uses both, so one should be converted to the other (e.g. "Build Selection", "Stop all currently running jobs" - both in the main toolbar).

Thanks

Reproducible: Always

Steps to Reproduce:
-
Actual Results:  
-

Expected Results:  
-

openSUSE 12.1 x86-64, KDE 4.7.2, Qt 4.7.4.
Comment 1 Milian Wolff 2012-05-04 12:18:07 UTC
confirmed, should be easy to change - any takers? patches welcome on git.reviewboard.kde.org!
Comment 2 Andrea Scarpino 2013-01-24 22:31:54 UTC
Git commit 01530060bc04455323af7fdf420e4da539b45494 by Andrea Scarpino.
Committed on 24/01/2013 at 23:30.
Pushed by scarpino into branch 'master'.

Consistent styles in tooltips
FIXED-IN: 4.6.0
REVIEW: 108492

M  +5    -6    debugger/breakpoint/breakpointwidget.cpp
M  +8    -8    debugger/framestack/framestackwidget.cpp
M  +7    -7    debugger/variable/variablewidget.cpp
M  +3    -4    plugins/appwizard/appwizardplugin.cpp
M  +2    -4    plugins/codeutils/codeutilsplugin.cpp
M  +2    -2    plugins/documentswitcher/documentswitcherplugin.cpp
M  +0    -1    plugins/documentview/kdevdocumentview.cpp
M  +0    -1    plugins/filemanager/filemanager.cpp
M  +1    -1    plugins/filetemplates/filetemplatesplugin.cpp
M  +5    -6    plugins/grepview/grepviewplugin.cpp
M  +0    -2    plugins/konsole/kdevkonsoleview.cpp
M  +19   -25   shell/debugcontroller.cpp
M  +7    -7    shell/documentcontroller.cpp
M  +18   -19   shell/mainwindow_p.cpp
M  +13   -7    shell/projectcontroller.cpp
M  +17   -17   shell/runcontroller.cpp
M  +1    -1    shell/sessioncontroller.cpp
M  +3    -5    shell/sourceformattercontroller.cpp
M  +1    -1    util/environmentconfigurebutton.cpp

http://commits.kde.org/kdevplatform/01530060bc04455323af7fdf420e4da539b45494