Skip to content
This repository was archived by the owner on Jun 24, 2022. It is now read-only.

[1.10] Mobile: issues with the header buttons #2148

Closed
elena-zh opened this issue Jan 14, 2022 · 2 comments
Closed

[1.10] Mobile: issues with the header buttons #2148

elena-zh opened this issue Jan 14, 2022 · 2 comments
Assignees
@fairlighteth
Labels
app:CowSwap CowSwap app Bug Something isn't working Claim Issues related to 'Claim' feature Low Severity indicator for defects. It won't cause any major break-down of the system

Comments

@elena-zh
Copy link

elena-zh commented Jan 14, 2022 •
edited
Loading

  1. iOS 15.3: menu button has square bg boarders in 'active' and 'hovered' states. The same is for 'vCOW' button in the menu
    image.png
    image

  2. Network dropdown seems to overlap the profile button when screen width = [960-1030]
    image
@elena-zh elena-zh added app:CowSwap CowSwap app Medium Severity indicator. It causes some undesirable behavior, but the system is still functional Design Design related Bug Something isn't working Claim Issues related to 'Claim' feature labels Jan 14, 2022
@elena-zh elena-zh mentioned this issue Jan 14, 2022
Merged
@elena-zh elena-zh changed the title PR-2132: issues with the header button PR-2132/PR-2129: issues with the header button Jan 14, 2022
@elena-zh elena-zh mentioned this issue Jan 14, 2022
Merged
@elena-zh elena-zh mentioned this issue Jan 28, 2022
Merged
@elena-zh elena-zh changed the title PR-2132/PR-2129: issues with the header button Mobile: issues with the header buttons Jan 28, 2022
@elena-zh elena-zh changed the title Mobile: issues with the header buttons [1.10] Mobile: issues with the header buttons Feb 2, 2022
@fairlighteth fairlighteth mentioned this issue Feb 2, 2022
Merged
@fairlighteth
Copy link
Contributor

  • Mostly being addressed in Improve responsiveness of header elements. #2376.
  • Proposing to not fix point 1 (Safari/iOS). This is due to the use of filter: blur. Tried some debugging but not worth the effort for the time being, considering this effect will be removed/is temporary.

@elena-zh elena-zh added Low Severity indicator for defects. It won't cause any major break-down of the system and removed Medium Severity indicator. It causes some undesirable behavior, but the system is still functional Design Design related labels Feb 2, 2022
@elena-zh
Copy link
Author

elena-zh commented Feb 2, 2022

@biocom , thanks!
I changed the priority to low. But it would not close this issue as it might be reported by end users.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@fairlighteth fairlighteth

Labels
app:CowSwap CowSwap app Bug Something isn't working Claim Issues related to 'Claim' feature Low Severity indicator for defects. It won't cause any major break-down of the system
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@fairlighteth @elena-zh