Hacker News Board Hacker News, browsed like a classic message board
Live data from Hacker News

Show HN: Leaflet.pub – a web app for creating and sharing rich documents

news.ycombinator.com

Post reply on HN
Print view View original

31–40 of 61 posts

Re: Show HN: Leaflet.pub – a web app for creating and sharing rich documents

#33

Post by jpereira »

pazimzadeh wrote: post #32

I like the icons. Can you edit a button after adding it?

Unfortunately not yet, we haven't come up with a pattern we like for that yet, just added buttons quite recently. Right now the move is to delete and re-add, which I know is annoying. Will improve soon!

Re: Show HN: Leaflet.pub – a web app for creating and sharing rich documents

#34

Post by jpereira »

jofer wrote: post #17

Just as an FYI: "leaflet" is the name of a popular web mapping JS library. https://leafletjs.com/ May or may not be relevant for you (and is definitely a very different field/product), but if you're releasing client libraries/SDKs of any sort, it might be good to be aware of. There's a large ecosystem of plugin libraries named "leaflet-foo" or "foo-leaflet" etc in addition to the "main" one. If you start releasing an…

Leaflet looks like an amazing library. We only came across it and the naming collision after we got started, and so far haven't had too much confusion, but definitely a good idea to be aware when we do an SDK. Would love to make the confusion worse by including a map block in leaflet.pub via leafletjs someday though!

Re: Show HN: Leaflet.pub – a web app for creating and sharing rich documents

#39

Post by chrismorgan »

nsriv wrote: post #2

Still playing around with it but I have to say, I absolutely adore the side-scrolling feature!

Just one thing needs fixing, urgently: stop messing with the scrollbars! You should never take away the scrollbar unless you’re replacing it with some similar affordance (and even that’s iffy).

This really is a big deal. It leads to people flat-out missing that things are scrollable. Even after I noticed it was doing that in one of the demos, I missed the scrolled content in the other demo for a while.

This needs two changes: firstly, ditch the .no-scrollbar class; secondly, on your .overflow-y-scroll and .overflow-x-scroll, replace the value ‘scroll’ with ‘auto’. (It’s a case of bad naming, compounded by the advent of overlay scrollbars which make it easy to miss that it’s forcing scrollbars, not merely allowing scrollbars, and inconsistent with almost all of the rest of CSS in that a value named ‘auto’ is not the default. ‘scroll’ has only ever had approximately three rare but semi-legitimate use-cases, and at least two of them were rendered obsolete with the advent of the ‘scrollbar-gutter’ property. My personal opinion is that `overflow: scroll` should be officially deprecated, linters should error on it, and browsers should even consider replacing it with `overflow: auto` as an intervention.)

Re: Show HN: Leaflet.pub – a web app for creating and sharing rich documents

#40

Post by 4ndrewl »

jofer wrote: post #17

Just as an FYI: "leaflet" is the name of a popular web mapping JS library. https://leafletjs.com/ May or may not be relevant for you (and is definitely a very different field/product), but if you're releasing client libraries/SDKs of any sort, it might be good to be aware of. There's a large ecosystem of plugin libraries named "leaflet-foo" or "foo-leaflet" etc in addition to the "main" one. If you start releasing an…

My first thought too.
Post reply on HN