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

The loading icon is not at the same position as the checkbox it replaces #7724

Closed
danxuliu opened this issue Jan 7, 2018 · 8 comments
Closed
Labels
bug design Design, UI, UX, etc. feature: sharing good first issue Small tasks with clear documentation about how and in which place you need to fix things in. papercut Annoying recurring issue with possibly simple fix.
Milestone

Comments

@danxuliu
Copy link
Member

danxuliu commented Jan 7, 2018

In the Sharing tab of the details view of the Files app when a checkbox is clicked it is replaced by a working/loading icon while the operation is being processed in the server. However, the icon is not placed exactly at the same position as the checkbox it replaces.

In the screenshots below it is subtle, but it can be clearly noticed when clicking on the checkbox, as when the icon is shown the text next to it is moved (in the first case to the left, in the second case to the right).

Share link:
share-link-checkbox
share-link-loading

Allow editing:
share-link-editing-checkbox
share-link-editing-loading

I have noticed this in the Sharing tab, although I do not know if this technique of replacing the checkbox with the icon is used anywhere else (although in my opinion it should be, as it shows that the operation is in process while also ensuring that the user will not mess with it by clicking again on the checkbox before it has finished ;-) ).

@nextcloud/designers

@danxuliu danxuliu added bug design Design, UI, UX, etc. feature: sharing labels Jan 7, 2018
@danxuliu danxuliu added this to the Nextcloud 13 milestone Jan 7, 2018
@MorrisJobke MorrisJobke added the papercut Annoying recurring issue with possibly simple fix. label Jan 7, 2018
@jancborchardt jancborchardt added the good first issue Small tasks with clear documentation about how and in which place you need to fix things in. label Jan 9, 2018
@MorrisJobke
Copy link
Member

It's fine to fix this in 14 and maybe backport to stable13.

@MorrisJobke MorrisJobke modified the milestones: Nextcloud 13, Nextcloud 14 Jan 9, 2018
@skjnldsv
Copy link
Member

@MorrisJobke backporting isn't really necessary, this doesn't break the uid nor the ux!

@soumyanjaleemahapatra
Copy link

Can I take over this issue?

@skjnldsv
Copy link
Member

@soumyanjaleemahapatra go ahead ! :)

@AasthaGupta
Copy link
Member

@soumyanjaleemahapatra are you still working on this or can I take this up?

@skjnldsv
Copy link
Member

skjnldsv commented Feb 23, 2018

If one of you is taking over this issue, please fix this in a global way. So that every checkbox using the icon-loading-small class has the same design. Do not fix only the sharing sidebar checkboxes :)

@soumyanjaleemahapatra
Copy link

@AasthaGupta Please take over the issue if you can. Thanks!

@nextcloud-bot nextcloud-bot added the stale Ticket or PR with no recent activity label Jun 20, 2018
@MorrisJobke
Copy link
Member

Fixed by the migration to the dropdown:

bildschirmfoto 2018-07-24 um 16 59 21

#10218

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug design Design, UI, UX, etc. feature: sharing good first issue Small tasks with clear documentation about how and in which place you need to fix things in. papercut Annoying recurring issue with possibly simple fix.
Projects
None yet
Development

No branches or pull requests

7 participants