overflow scroll ?

[nweldev]

On the demo, or when starting a new project using deckdeckgo-starter, the first slide looks like this

This is due to an overflow: scroll introduced by 12a843d

deckdeckgo/studio/src/global/theme/editor/editor-deck-deck.scss

Lines 15 to 19 in e27acdf

	deckgo-deck h1, deckgo-deck h2, deckgo-deck h3, deckgo-deck section, deckgo-deck ol, deckgo-deck ul, deckgo-deck deckgo-reveal-list, deckgo-deck deckgo-highlight-code, deckgo-deck deckgo-lazy-img[slot] {
	margin: 16px 0;
	overflow: scroll;
	outline: none;
	}

browser: Chrome 78.0.3876.0 & Firefox 69.0b15
version: starter 1.0.0-beta.22

[peterpeterparker]

Raaah ptain :(

Windows or Linux?

On my Mac, Chrome or Firefox, starter-kit and website are both ok.

P.S.: Thx for opening an issue about it 👍

[nweldev]

Linux

[peterpeterparker]

Thx for the feedback 👍

I guess I just have to change the scroll to auto and it should be solved and still compatible with our upcoming editor. I'll meet @nmattia later today, I will test it first on his linux laptop

[nweldev]

I'm confused ... how is it related to the OS?

[peterpeterparker]

Me too not sure why but I think I already faced that kind of same weird behavior with another element a couple of weeks/months ago. No scrollbars were displayed on my Mac but some were displayed on Nicolas' linux machine.

I'm just guessing that it might be the case again.

Anyway I guess auto is the solution, just wand to be sure I'll fix it once for all ;)

[nweldev]

That's really weird indeed 😄 I'd be very interessed to know the root cause if you find it!

[peterpeterparker]

Might actually be an "OSX issue" or the fact that on OSX scrollbars are (per default) most of the time hidden by the system, that's why I thought it was all good probably 😒

See screenshots when I "force" OSX to display the scrolls

[nweldev]

This is good to know thx!

[peterpeterparker]

auto updated in website, starter and studio. still need to double check but should be good

[peterpeterparker]

Tested on Linux, looks good now

But I would be happy if you could double check @noelmace

Thx for reporting the issue

[nweldev]

LGTM 👌 Thx