Review Request 108212: [1/3] Refactor EnvironmentSelectionWidget.

Ivan Shapovalov intelfx100 at gmail.com
Thu Dec 5 14:00:57 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/108212/
-----------------------------------------------------------

(Updated Dec. 5, 2013, 2 p.m.)


Review request for KDevelop.


Changes
-------

More issues fixed.


Repository: kdevplatform


Description
-------

Feature-wise this gets rid of dependency on default KConfigXT value being "default" and allows explicitly specifying to use a default environment group whatever it will be.

Now an empty value shall be used by default, which has meaning of "use whatever is set by default at runtime".
Also, a special "Use default" line is now always added to the combo-box which maps to the empty config value, so user won't see empty entries in the combo-box.


Diffs (updated)
-----

  plugins/executescript/scriptappconfig.cpp 370231a 
  plugins/execute/nativeappconfig.cpp 6f2dd8c 
  util/CMakeLists.txt 87384bd 
  util/environmentconfigurebutton.cpp b95e536 
  util/environmentgrouplist.h 4981b73 
  util/environmentgrouplist.cpp cec71bd 
  util/environmentselectionmodel.h PRE-CREATION 
  util/environmentselectionmodel.cpp PRE-CREATION 
  util/environmentselectionwidget.h 0784ba1 
  util/environmentselectionwidget.cpp a198f32 

Diff: http://git.reviewboard.kde.org/r/108212/diff/


Testing
-------

Manual testing.


Thanks,

Ivan Shapovalov

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20131205/e00f2c17/attachment-0001.html>


More information about the KDevelop-devel mailing list