Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NMC-127 - File action button behaviour for different viewport #28834

Closed
wants to merge 11 commits into from

Conversation

TSI-kavitasonawane
Copy link
Contributor

@TSI-kavitasonawane TSI-kavitasonawane commented Sep 14, 2021

File action button behaviour changes:
We now show more single actions depending on the viewport size in the table header beginning from the first option in the dropdown.
We add a new option called "Cancel" (ger: Abbrechen") which will remove any selection (just remove selection, not the files of course).
We right align the Actions button.
We will show every option in the action button dropdown. It doesnt matter which option is also shown in the table header.
We add a new label called "All".
on S: we hide the label "All" according to the screen design

Screenshot after changes:
file action

apps/files/js/app.js Outdated Show resolved Hide resolved
@juliushaertl
Copy link
Member

Some screenshots for before/after would be nice to make this easier to review for our design team :)

@juliushaertl juliushaertl requested review from a team, PVince81, julien-nc and skjnldsv and removed request for a team October 4, 2021 06:07
Signed-off-by: Kavita Sonawane <[email protected]>
Copy link
Contributor

@artonge artonge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like this change a lot, but I have some design feedback:

There is a flash when selecting or deselecting an element

Peek 2021-10-04 12-54

Can we have some additional style on hover?

Some ugly ideas:

Screenshot from 2021-10-04 12-52-06
OR
Screenshot from 2021-10-04 12-52-23

Can we align "All" and "1 file (164 B)"

Screenshot from 2021-10-04 12-46-17

Can we make the "Cancel" button different to highlight that it is not an action like the others?

Something like that with the cross also styled in red:

Screenshot from 2021-10-04 12-58-50

Signed-off-by: Kavita Sonawane <[email protected]>
@TSI-kavitasonawane
Copy link
Contributor Author

I like this change a lot, but I have some design feedback:

There is a flash when selecting or deselecting an element

Peek 2021-10-04 12-54

Can we have some additional style on hover?

Some ugly ideas:

Screenshot from 2021-10-04 12-52-06 OR Screenshot from 2021-10-04 12-52-23

Can we align "All" and "1 file (164 B)"

Screenshot from 2021-10-04 12-46-17

Can we make the "Cancel" button different to highlight that it is not an action like the others?

Something like that with the cross also styled in red:

Screenshot from 2021-10-04 12-58-50

@artonge I have done all the changes which you suggested. Please check it.

@artonge
Copy link
Contributor

artonge commented Oct 5, 2021

@artonge I have done all the changes which you suggested. Please check it.

Thanks, this looks much better already :)

Three new nitpicks:

  1. The trash icon is red on hover, can we prevent that to make it more persistent with other actions ? Or maybe go further and make it red when the label is hovered too. Maybe make the label red too.

Screenshot from 2021-10-05 11-04-05

  1. Should we move the Cancel action outside of the three dots menu and have it always accessible ?

  2. Should we only list the hidden actions in the three dots menu instead of listing them all ?

Screenshot from 2021-10-05 11-10-39

Maybe we should @jancborchardt for some proper design inputs :)

@skjnldsv skjnldsv added this to the Nextcloud 22.2.2 milestone Oct 19, 2021
Signed-off-by: Kavita Sonawane <[email protected]>
@TSI-kavitasonawane
Copy link
Contributor Author

TSI-kavitasonawane commented Oct 20, 2021

@artonge I have done all the changes which you suggested. Please check it.

Thanks, this looks much better already :)

Three new nitpicks:

  1. The trash icon is red on hover, can we prevent that to make it more persistent with other actions ? Or maybe go further and make it red when the label is hovered too. Maybe make the label red too.

@artonge I have done the changes which you suggested for trash icon.

Screenshot from 2021-10-05 11-04-05

  1. Should we move the Cancel action outside of the three dots menu and have it always accessible ?

We can't do this because we don't have space for Cancel action in small screen. Please refer below screenshot for the same
Table Header Functions S@2x

  1. Should we only list the hidden actions in the three dots menu instead of listing them all ?
    As per requirement we have to show every option in the action button dropdown. It doesnt matter which option is also shown in the table header. @jancborchardt Please confirm this.
    Screenshot from 2021-10-05 11-10-39

Maybe we should @jancborchardt for some proper design inputs :)

@juliushaertl
Copy link
Member

Maybe we should @jancborchardt for some proper design inputs :)

cc @nimishavijay 😉

@nimishavijay
Copy link
Member

Really nice work! Suggestions from my end:

image

This button style looks better for actions, because with an underline it looks like a link

The trash icon is red on hover, can we prevent that to make it more persistent with other actions ? Or maybe go further and make it red when the label is hovered too. Maybe make the label red too.

I agree with removing the red for consistency, in any action menu it is not red either :)

Should we move the Cancel action outside of the three dots menu and have it always accessible ?

I agree, it is a good idea.
Many other softwares have a different pattern for the Cancel action, it is in the "select all" checkbox.
Clicking on the select all checkbox when only some items are selected will unselect everything. That way we don't need to have a separate action called "Cancel". I've pasted some examples here. What do you think?

Gmail

gmail-select

Dropbox

dropbox-select

If we are keeping the Cancel action, can we get rid of the red colour? It makes it look like a much more dangerous action than it actually is.

Should we only list the hidden actions in the three dots menu instead of listing them all ?

Yes, agreed, we don't have to repeat actions.

More suggestions:

  • "Delete" action is always the last item in the menu, we follow this UI pattern everywhere.
  • We don't need the "Actions" label, just the 3 dot menu is enough as we are already showing actions next to it

Looks super cool :)
cc @jancborchardt when he sees this

Signed-off-by: Kavita Sonawane <[email protected]>
Signed-off-by: Kavita Sonawane <[email protected]>
Signed-off-by: Kavita Sonawane <[email protected]>
@jancborchardt
Copy link
Member

Agree with all of @nimishavijay’s feedback above – @TSI-kavitasonawane anything remaining from those items? :)

@blizzz blizzz added this to the Nextcloud 24 milestone Nov 10, 2021
@blizzz
Copy link
Member

blizzz commented Nov 10, 2021

changing the milestone to 24 for the PR is against master, a feature and we branch off 23 very soon.

@skjnldsv skjnldsv mentioned this pull request Mar 24, 2022
@blizzz blizzz mentioned this pull request Mar 31, 2022
This was referenced Apr 7, 2022
@blizzz blizzz modified the milestones: Nextcloud 24, Nextcloud 25 Apr 21, 2022
This was referenced Aug 12, 2022
This was referenced Aug 24, 2022
This was referenced Sep 6, 2022
@skjnldsv skjnldsv mentioned this pull request Sep 15, 2022
This was referenced Sep 20, 2022
@blizzz blizzz modified the milestones: Nextcloud 25, Nextcloud 26 Sep 22, 2022
@blizzz blizzz mentioned this pull request Feb 1, 2023
@skjnldsv skjnldsv mentioned this pull request Feb 23, 2023
@blizzz blizzz mentioned this pull request Mar 7, 2023
@blizzz blizzz modified the milestones: Nextcloud 26, Nextcloud 27 Mar 9, 2023
@skjnldsv skjnldsv closed this Apr 11, 2023
@juliushaertl
Copy link
Member

@skjnldsv All covered in #37642 right? Just to keep a reference here as well

@skjnldsv
Copy link
Member

yes!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants