question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Popup should have a min-height

See original GitHub issue

The close button overflows the popup container if the container is empty:

grafik

var marker = new L.Marker(latlng).addTo(map);
marker.bindPopup("");

Issue Analytics

  • State:closed
  • Created a year ago
  • Comments:6 (5 by maintainers)

github_iconTop GitHub Comments

2reactions
Falke-Designcommented, Apr 11, 2022

It’s not important but I think it would be good to have it, because it is clearly an UI topic which comes from Leaflet. And it is not a big thing it is one css class which needs min-height: 24px

2reactions
Malvozcommented, Apr 11, 2022

@MassiGy You could utilize the :empty pseudo-class. Also, the min-height doesn’t have to be that excessive. Use the smallest min-height that would fix the issue.

@Falke-Design Is it really necessary to fix this? Seems like an enhancement for development only. If popups are empty in production that’s the wrong use of a popup.

Edit: OTOH there could be really small content in popups, that this would fix. In which case :empty is not a good solution.

Read more comments on GitHub >

github_iconTop Results From Across the Web

Resize popup Window to min. height and width then dragging ...
Resize popup Window to min. height and width then dragging to below size should not possible · Ask Question. Asked 9 years, 4...
Read more >
how to set the min width and min height for popups throughout ...
I have read the below sample code on how to set the minimum popup width and height for a given popup windows. However,...
Read more >
minHeight - ui.popup - Webix Docs
This page contains minHeight documentation to help in learning the library. ... If more (free) space is available - the view will take...
Read more >
ASPxPopupControlBase.MinHeight Property
To specify the minimum height to which an end-user can resize the particular window, use the PopupWindow.MinHeight property. #Example. This example illustrates ...
Read more >
jQuery UI Dialog minHeight Option - GeeksforGeeks
Dialog minHeight option is used to set the minimum height that can be set to the dialog box. By default, value is 150....
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found