Skip to content
This repository has been archived by the owner on Jun 3, 2024. It is now read-only.

QMAPS-2159 fix error messages height on mobile #1104

Merged
merged 2 commits into from
Jun 1, 2021
Merged

Conversation

xem
Copy link
Contributor

@xem xem commented Jun 1, 2021

Description

error panels on mobile were too tall
now they're as tall as the text
NB: this fix shouldn't impact the display of these panels when there's no error

Screenshots

image
image

@xem xem requested a review from bbecquet June 1, 2021 08:17
src/panel/NoResultPanel.jsx Outdated Show resolved Hide resolved
src/panel/category/CategoryPanel.jsx Outdated Show resolved Hide resolved
@xem xem requested a review from bbecquet June 1, 2021 15:42
@bbecquet
Copy link
Contributor

bbecquet commented Jun 1, 2021

@xem

thanks for the explanations, I hadn't use this mechanism until today and it wasn't super clear for me how to use it properly.

You're right, this feature is powerful but confuse and hard to discover. But it's great when our existing components can be used as-is to bring enhancements like this 👍

@xem xem merged commit e2b54b6 into master Jun 1, 2021
@amatissart amatissart deleted the QMAPS-2159-fit-content branch June 29, 2021 15:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants