smashingmagazine / redesign

Reporting bugs and weird issues on the shiny new Smashing Magazine.
https://next.smashingmagazine.com
41 stars 2 forks source link

Article header positioning bug in Firefox 57 (Responsive, 1008px+) #305

Closed chrisheninger closed 6 years ago

chrisheninger commented 6 years ago

- Do you want to request a feature or report a bug? Bug

- What is the current behavior? .article__header on https://www.smashingmagazine.com/design-systems-book/ in Firefox are positioned incorrectly when the viewport is sized at over 1008px.

screen shot 2017-11-28 at 11 05 01 am

- Please mention your operating system version and the version of your browser Mac OS X (latest) with Firefox 57 (latest)

vitalyfriedman commented 6 years ago

Unfortunately I couldn't reproduce it at all @chrisheninger — could you try again?..

chrisheninger commented 6 years ago

Still broken in Firefox for me. My assumption about .article__header was wrong– after taking a closer look it's something to do with the way Firefox is handling the :before block and the paragraph that is supposed to be wrapping around it.

screen shot 2017-11-30 at 2 47 19 pm

Adding a display: none; to the .drop-caps fixes it, but you lose the cute illustration. :(

screen shot 2017-11-30 at 2 54 38 pm
vitalyfriedman commented 6 years ago

Very strange. Still trying to fix the bug. Seems to be a weird one @chrisheninger

chrisheninger commented 6 years ago

Good news... it's fixed! Not sure if something happened on your end or if FF updating to 57.0.1 fixed it... but looks nice now. :)

screen shot 2017-12-01 at 9 13 20 pm

I'll let you handle closing. 👍

vitalyfriedman commented 6 years ago

Hummm... Not so happy about magic solutions to problems... Closing for now, but will keep an eye in case it appears again. Thanks for reporting the issue, dear @chrisheninger!