search

nextcloud / deck

🗂 Kanban-style project & personal management tool for Nextcloud, similar to Trello
https://apps.nextcloud.com/apps/deck
GNU Affero General Public License v3.0
1.21k stars 275 forks source link

Move edit/delete stack icons to the popover menu #170

Closed juliusknorr closed 7 years ago

juliusknorr commented 7 years ago

https://github.com/nextcloud/deck/pull/121#issuecomment-304907357

artemanufrij commented 7 years ago

@jancborchardt like this: screenshot_20170530_230245

pixelipo commented 7 years ago

@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?

jancborchardt commented 7 years ago

@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 commented 7 years ago

@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/ ;)

artemanufrij commented 7 years ago

@jancborchardt Cool! I plan to visit NC conference