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

Avoid misunderstanding of the "Copy to sharing cart" icon as inactive #68

Closed
abias opened this issue Sep 25, 2020 · 3 comments
Closed

Comments

@abias
Copy link
Contributor

abias commented Sep 25, 2020

Hi Don, dear Praxis team,

while upgrading to Moodle 3.9 and Sharing Cart 3.8 Release 10, I saw that you have pulled the "Copy to sharing cart" functionality out from the activity edit menu and have changed it to a dedicated icon:

image

While I appreciate this change, I am afraid that the fact that you are showing the icon with a very light tone of the brand color and are changing it to a solid color only while hovering can produce misunderstandings for users who think that the icon is disabled. And, because of the low contract, it can trigger accessibility problems as well, of course.

I would like to ask you to rethink this choice and use a more solid color when not hovered.
Ideally, use the group icons of Moodle core as example:

image

They have a solid color when not hovered, but they also have a small hovering effect.

Thanks for your consideration.

Cheers,
Alex

@Kathrin84
Copy link
Contributor

Kathrin84 commented Sep 29, 2020

He there,

Alex asked me to add my insights.

The reason for this appearance is this rule:

.add-to-sharing-cart {
    opacity: 0.3;
}

.add-to-sharing-cart {
opacity: 0.3;
}

In our individual theme RAW SCSS, we changed this back to full opacity. Even with full opacity, there is still a hovering effect.

Best, Kathrin

@abias
Copy link
Contributor Author

abias commented Jun 17, 2021

I would like to point out that this problem still persists in Version 3.9, release 3. But I think that it should be easy to fix.

donhinkelman pushed a commit that referenced this issue Oct 12, 2022
… shake

Changed spinner to shaking basket icon
donhinkelman pushed a commit that referenced this issue Oct 12, 2022
#68 - Added indications on hover and cancel/submit
@AraPraxis
Copy link
Contributor

CSS, styling and functionality has been updated in Moodle 4.0+ releases.

We do not intent to make any updates to sharing cart prior to Moodle 4.0 - However we do follow the Moodle releases and are willing to fix any security risks regarding this plugin.
https://moodledev.io/general/releases

Kind regards,
Anders Rasmussen

Praxis Team

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

No branches or pull requests

4 participants