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

Favorite action missing visual feedback #1903

Closed
PVince81 opened this issue Sep 5, 2019 · 2 comments
Closed

Favorite action missing visual feedback #1903

PVince81 opened this issue Sep 5, 2019 · 2 comments
Labels

Comments

@PVince81
Copy link
Contributor

PVince81 commented Sep 5, 2019

Steps to reproduce

  1. Create a folder
  2. Approach the cursor to the star
  3. Click it
  4. Wait

Expected behaviour

When the cursor approaches it, it should transform to a hand. Or any other visual feedback appreciated to indicate that the star is clickable.
When clicking, a visual feedback should aknowledge the action, maybe spinner while the request is running, then eventually the star should turn yellow.

Actual behaviour

No visual feedback on hover.
No visual feedback while the request is running. It looks like nothing happens until suddenly the star turns yellow after a few seconds.

Versions

ownCloud Server 10.3.0 alpha2
Phoenix master 2f7b3d7

@PVince81 PVince81 added this to the Milestone 1: Phoenix for users milestone Sep 24, 2019
@PVince81 PVince81 added the Priority:p3-medium Normal priority label Sep 24, 2019
@marcus-herrmann
Copy link
Contributor

marcus-herrmann commented Oct 21, 2019

No visual feedback while the request is running. It looks like nothing happens until suddenly the star turns yellow after a few seconds.

I would change the star's state right after the click, regardless of the request this click causes (independent from the actual favorite state). This would improve the perceived performance. Right now its display as starred is dependent from a state (starred) that's coming from a successful server call:

<oc-star class="uk-display-block" @click.native.stop="toggleFileFavorite(item)" :shining="item.starred" />

Additionally we should consider if we throw a notification on sucessful (de-)favoriting.

No visual feedback on hover.

A cursor: pointer ("Mousehand") in CSS would not work with touch. Since it will become an actual button after the related PR is merged, the most robust solution would be to give the star button looks.

@pascalwengerter
Copy link
Contributor

Fixed via #5000 right now on the a11y-swarming branch, coming to master soon

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

No branches or pull requests

3 participants