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

fix(search): lateral btn over autocomp results (mobile) #446

Merged
merged 3 commits into from
Jan 10, 2025

Conversation

azarz
Copy link
Member

@azarz azarz commented Jan 6, 2025

cf #444

avant
398396967-1ecb0c74-fe1b-4439-96e8-963cf67c80b7

après
image

@azarz azarz requested a review from elias75015 January 6, 2025 15:17
@azarz azarz self-assigned this Jan 6, 2025
@azarz azarz requested a review from IGNFhc January 10, 2025 08:47
@elias75015 elias75015 added the bug Something isn't working label Jan 10, 2025
@elias75015 elias75015 added this to the V2 milestone Jan 10, 2025
@elias75015
Copy link
Contributor

Oups j'avais zappé.

👍 ça corrige bien le problème initial.

Petit bug en revanche :
image

Je réduis la fenêtre, et le bouton du layerSwitcher passe sous celui du menu des outils. J'imagine qu'il fadrait le redécaler vers le bas ?
image

Copy link
Contributor

@elias75015 elias75015 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

voir commentaire ci-dessus

@azarz
Copy link
Member Author

azarz commented Jan 10, 2025

Oups j'avais zappé.

👍 ça corrige bien le problème initial.

Petit bug en revanche : image

Je réduis la fenêtre, et le bouton du layerSwitcher passe sous celui du menu des outils. J'imagine qu'il fadrait le redécaler vers le bas ? image

C'est aussi sur main

@azarz
Copy link
Member Author

azarz commented Jan 10, 2025

Je réduis la fenêtre, et le bouton du layerSwitcher passe sous celui du menu des outils. J'imagine qu'il fadrait le redécaler vers le bas ?

Corrigé

@azarz azarz requested a review from elias75015 January 10, 2025 11:19
Copy link
Contributor

@elias75015 elias75015 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏 👏

@azarz azarz merged commit 0490a2c into main Jan 10, 2025
@azarz azarz deleted the fix/lateral-buttons-over-autocomp-results branch January 10, 2025 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants