Skip to content

show search bar again#17

Open
muelli wants to merge 1 commit into
thomersch:masterfrom
muelli:patch-2
Open

show search bar again#17
muelli wants to merge 1 commit into
thomersch:masterfrom
muelli:patch-2

Conversation

@muelli

@muelli muelli commented May 31, 2022

Copy link
Copy Markdown
Contributor

Somehow the map goes full screen and hides the search bar. This makes it slightly better, but it doesn't fully solve the problem, since there are some pixels overlapping. But at least the search can be used again.

Somehow the map goes full screen and hides the search bar. This makes it slightly better, but it doesn't fully solve the problem, since there are some pixels overlapping. But at least the search can be used again.
@thomersch

Copy link
Copy Markdown
Owner

True, maybe that's some change of leaflet's behaviour. 50px doesn't looks right to me, though. It looks like the bar is roughly 70px tall. Maybe you could set a fixed height for the top bar and then have the same value for the margin? Alternatively we could put both elements into a flexbox container and let the browser figure that one out.

@muelli

muelli commented Dec 1, 2022

Copy link
Copy Markdown
Contributor Author

height:3rem;

3rem seems to be the definition.

Changing #bigmap to 4rem makes it look reasonably nice.

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

Successfully merging this pull request may close these issues.

2 participants