I opened #5131 and put a number of changes to error pages to see which of them 
are acceptable. The discussion there however has been about making more 
changes. I guess that means nobody objects to the ones I made and we can have 
them merged.

This PR changes the font of error pages to a default system ui font. Bootstrap 
does the same thing except also adds fallback fonts for older browsers that 
don't understand `font-family: system-ui`. We don't need a serif font 
for a text this short because we don't use serif fonts even for longer 
texts on the website.

Addresses this from #3532:
> It also looks incredibly ugly, with loads of whitespace and the default 
browser font.

![image](https://github.com/user-attachments/assets/3e20f2f8-d3d9-404f-b403-c9f1eb338360)

You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/5277

-- Commit Summary --

  * Use system font on error pages

-- File Changes --

    M app/assets/stylesheets/errors.scss (4)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/5277.patch
https://github.com/openstreetmap/openstreetmap-website/pull/5277.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5277
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/5...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to