[Django] #34102: Admin list view `word-break: break-word;`
Django
-------------------------------------------+------------------------
Reporter: Collin Anderson | Owner: nobody
Type: Bug | Status: new
Component: contrib.admin | Version: dev
Severity: Normal | Keywords:
Triage Stage: Unreviewed | Has patch: 0
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 0 |
-------------------------------------------+------------------------
Hi All,
15682cb0034242fcdfe333cf399d62ab70c3a978 /
https://github.com/django/django/pull/16164 (Fixed wrapping of long model
names in the admin, Oct 10, 2022) put `word-break: break-word;` on all
`td, th` table cells in the entire admin.
That kinda messes up admin list view when there's a lot of columns, where
it's now breaking data mid-word or mid-email, etc. It might "break" other
things too. I'd suggest sticking to the browser default `word-break` for
most tables, and maybe limit `word-break: break-word;` to just the index
page.
Thanks,
Collin
--
Ticket URL: <https://code.djangoproject.com/ticket/34102>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
Django
Reporter: Collin Anderson | Owner: nobody
Type: Bug | Status: new
Component: contrib.admin | Version: dev
Keywords: | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Changes (by Mariusz Felisiak):
* severity: Normal => Release blocker
* stage: Unreviewed => Accepted
Comment:
Thanks for the report! Visual regression in
15682cb0034242fcdfe333cf399d62ab70c3a978.
> I'd suggest sticking to the browser default `word-break` for most
tables, and maybe limit `word-break: break-word;` to just the index page.
Sounds good. Would you like to prepare a patch?
--
Ticket URL: <https://code.djangoproject.com/ticket/34102#comment:1>
Django
---------------------------------+------------------------------------
Reporter: Collin Anderson | Owner: nobody
Type: Bug | Status: new
Component: contrib.admin | Version: dev
Severity: Release blocker | Resolution:
Keywords: | Triage Stage: Accepted
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
---------------------------------+------------------------------------
* has_patch: 0 => 1
Comment:
Ok here's a patch: https://github.com/django/django/pull/16190
--
Ticket URL: <https://code.djangoproject.com/ticket/34102#comment:2>
Django
Reporter: Collin Anderson | Owner: Collin
| Anderson
Type: Bug | Status: assigned
Component: contrib.admin | Version: dev
Severity: Release blocker | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Changes (by Mariusz Felisiak):
* owner: nobody => Collin Anderson
* status: new => assigned
--
Ticket URL: <https://code.djangoproject.com/ticket/34102#comment:3>
Django
-------------------------------------+-------------------------------------
Reporter: Collin Anderson | Owner: Collin
| Anderson
Component: contrib.admin | Version: dev
Severity: Release blocker | Resolution: fixed
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
* status: assigned => closed
* resolution: => fixed
Comment:
In [changeset:"78470043ae4b8def7914badcd05bb1877c8a0aa4" 7847004]:
{{{
#!CommitTicketReference repository=""
revision="78470043ae4b8def7914badcd05bb1877c8a0aa4"
Fixed #34102 -- Limited wrapping long names to admin dashboard.
Regression in 15682cb0034242fcdfe333cf399d62ab70c3a978.
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/34102#comment:4>