[Differential] [Request, 168 lines] D4395: File template dialog: several small UI/UX improvements

Friedrich W. H. Kossebau noreply at phabricator.kde.org
Wed Feb 1 14:37:37 UTC 2017


kossebau created this revision.
kossebau added a reviewer: KDevelop.
Restricted Application added a subscriber: kdevelop-devel.

REVISION SUMMARY
  Small improvements collected in the last weeks, for
  review in one go, but possibly to be split up in commits,
  also some for master and some for 5.1.
  
  - turn newclass.ui & testcases.ui into QFormLayout + fix bad automatic tab order in testcases.ui + "Inheritance:" label now properly aligned with lineedit + remove unneeded extra layout layers
  - workaround for broken focus handling in KEditListWidget on "Add" button clicks or "Remove" with last item
  - use generic "generated content" instead of "new class" in pages which are also used for non-class templates
  - use i18np instead of static "Output file(s)" on output page
  - use ICore::self()->projectController()->projectsBaseDirectory() as fallback starting folder, not QDir::homePath()
  - add a placeholder text to class identifier lineedit which hints that namespace can be entered there as well (was not obvious to me :) )

REPOSITORY
  R33 KDevPlatform

BRANCH
  pimpFileTemplateDialog

REVISION DETAIL
  https://phabricator.kde.org/D4395

AFFECTED FILES
  plugins/filetemplates/classidentifierpage.cpp
  plugins/filetemplates/classmemberspage.cpp
  plugins/filetemplates/filetemplatesplugin.cpp
  plugins/filetemplates/outputpage.cpp
  plugins/filetemplates/testcasespage.cpp
  plugins/filetemplates/ui/licensechooser.ui
  plugins/filetemplates/ui/newclass.ui
  plugins/filetemplates/ui/outputlocation.ui
  plugins/filetemplates/ui/testcases.ui

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: kossebau, #kdevelop
Cc: kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20170201/e8f57cbd/attachment.html>


More information about the KDevelop-devel mailing list