fix: stop revealing elements on hover if hover is not supported #3191
No reviewers
Labels
No Label
area/internal-code
changes requested
confirmed
dependencies
duplicate
good first issue
help wanted
hosting
invalid
kind/bug
kind/feature
question
wontfix
No Milestone
No project
No Assignees
3 Participants
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: vikunja/frontend#3191
Loading…
Reference in New Issue
No description provided.
Delete Branch "danstewart/vikunja-frontend:danstewart/fix-ios-double-tap"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Fixes #3162
A little clunky but works fine from my testing.
The only weird thing I've encountered is on mobile picking a list on the side menu doesn't auto collapse, but it's doing that on
main
so not related to these changes.Hi danstewart!
Thank you for creating a PR!
I've deployed the changes of this PR on a preview environment under this URL: https://3191-danstewart-fix-ios-double-tap--vikunja-frontend-preview.netlify.app
You can use this url to view the changes live and test them out.
You will need to manually connect this to an api running somehwere. The easiest to use is https://try.vikunja.io/.
Have a nice day!
@ -350,2 +350,4 @@
opacity: 1;
}
@media(hover: hover) and (pointer: fine) {
How is the browser support for this?
Pretty good:
https://caniuse.com/css-media-interaction
perfect
@ -162,3 +162,3 @@
.list-card:hover .favorite {
.list-card .favorite {
opacity: 1;
Aren't all the
opacity: 1
statements outside the media queries now redundant?Good catch, I was duplicating a few rules as I kept the original
opacity: 0
then immediately set it back toopacity: 1
.I've removed the duplicates now.
Looks like this now hides the favorite icons on desktop:
vs on try:
@konrad Odd, it works fine for me 🤔
Which browser and device are you using? I've tried on Chrome and Firefox.
Tested in Firefox and Chromium on NixOS. Tested with the review link above.
The favorite icons only show up when hovering, not by default (as they do on try).
Sorry, I was being dumb.
Should be sorted now - thank you.
@ -496,0 +506,4 @@
opacity: 1;
}
.list-menu:hover .favorite {
Can you unify these two rules to something like
@ -367,3 +363,2 @@
&:hover .handle {
opacity: 1;
@media(hover: hover) and (pointer: fine) {
same for unifying these rules here as well.
Addressed those last two comments. 🙂
Thanks!