D11214: Don't let the selection text overlap the Update button

2018-03-12 Thread Nathaniel Graham
ngraham abandoned this revision. ngraham added a comment. Abandoning in favor of D11218 , which is a better approach. REPOSITORY R134 Discover Software Store REVISION DETAIL https://phabricator.kde.org/D11214 To: ngraham, #discover_software_store,

D11214: Don't let the selection text overlap the Update button

2018-03-11 Thread Nathaniel Graham
ngraham added a comment. So you prefer this one to D11218 ? Note that we do lose the "n updates not selected" text with this approach. REPOSITORY R134 Discover Software Store BRANCH never-overlap-update-button (branched from master) REVISION DETAIL

D11214: Don't let the selection text overlap the Update button

2018-03-10 Thread Nathaniel Graham
ngraham added a comment. Possibly/likely superseded by D11218 . REPOSITORY R134 Discover Software Store REVISION DETAIL https://phabricator.kde.org/D11214 To: ngraham, #discover_software_store, apol Cc: plasma-devel, ZrenBot, lesliezhai, ali-mohamed,

D11214: Don't let the selection text overlap the Update button

2018-03-10 Thread Nathaniel Graham
ngraham created this revision. ngraham added a reviewer: Discover Software Store. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. ngraham requested review of this revision. REVISION SUMMARY By removing the "n unsepected" text (which