Deck: Move edit/delete stack icons to the popover menu
juliushaertl
All 5 comments
@jancborchardt like this:
artemanufrij
on 30 May 2017
@jancborchardt referring to your comment in #121 - I definitely agree with it (better accessibility, lower learning curve, cleaner UI), but I made a PR #173 which deprecates of of the two existing icons (Edit). Should that single (Delete) icon still be moved to the popover?
@juliushaertl you probably have plans for more actions related to stack (Move, Copy, Archive, Archive All Cards...), so popover definitely makes sense, but should we wait until we populate it more?
pixelipo
on 31 May 2017
@pixelipo nice, good call with making rename more seamless! :)
A single icon should never be in a popover, but please do add a tooltip to it which appears on hover (using the JS tooltip function). But yeah as you say maybe it makes most sense to do a popover right away when more actions come in.
jancborchardt
on 1 Jun 2017
@pixelipo @artemanufrij btw, if you folks come to Berlin some time, for example for our Nextcloud conference – we should make a bike tour! https://www.reddit.com/r/FixedGearBicycle/comments/6eib4c/breakbrake17_transfer_raw_cinelli_gazzetta/ ;)
jancborchardt
on 1 Jun 2017
@jancborchardt Cool! I plan to visit NC conference
artemanufrij
on 1 Jun 2017
Related issues
Dubidubiduu
·
3Comments
RobertZenz
·
4Comments
reox
·
4Comments
funky123
·
4Comments
desperateCoder
·
3Comments
Most helpful comment
@jancborchardt Cool! I plan to visit NC conference