[openstreetmap/openstreetmap-website] Force word wrap in tag table (PR #4713)

Anton Khorev notifications at github.com
Wed Apr 17 14:42:03 UTC 2024


Fixes #4709, kind of.

Aside from differences in browser behavior, there's also a decision by the Bootstrap developers. [They disable `.text-break` in rtl mode](https://getbootstrap.com/docs/5.3/utilities/text/#word-break). They say that rtl is most likely Arabic and word breaks in Arabic are wrong therefore disable word breaks for any rtl. But in our case any long words in tags table are likely not Arabic but rather urls or something similar.

Here I'm putting the css properties that are normally in Bootstrap's `.text-break` to our custom css. This will restore word breaks in rtl tag tables, occasionally with "wrong" Arabic word breaks.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Force word wrap in tag table

-- File Changes --

    M app/assets/stylesheets/common.scss (2)
    M app/views/browse/_tag_details.html.erb (2)

-- Patch Links --

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

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

Message ID: <openstreetmap/openstreetmap-website/pull/4713 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240417/732e0208/attachment.htm>


More information about the rails-dev mailing list