#35404: Admin fieldset multiple flexbox regressions
-------------------------------------+-------------------------------------
Reporter: minusf | Owner: nobody
Type: Bug | Status: new
Component: contrib.admin | Version: 4.2
Severity: Normal | Resolution:
Keywords: css admin fieldset | Triage Stage: Accepted
accessibility |
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 1
-------------------------------------+-------------------------------------
Changes (by Sarah Boyce):
* keywords: css admin fieldset => css admin fieldset accessibility
Comment:
Replying to [comment:9 minusf]:
> Regarding `help_text`, I think that with this small font `line-height:
normal` is not enough:
> {{{#!diff
> M django/contrib/admin/static/admin/css/base.css
> @@ -268,6 +268,7 @@ hr {
> .help, p.help, form p.help, div.help, form div.help, div.help li {
> font-size: 0.6875rem;
> color: var(--body-quiet-color);
> + line-height: 1.5;
> }
> ยท
> div.help ul {
> }}}
I don't think, taking into account the way the admin is already, this css
change clearly fixes an issue/bug.
I think a wider discussion on whether the fonts and spacing in the admin
is too small (with the accessibility team involved) is a good idea, but
out of scope of this ticket.
There are many small tweaks we can make to css and there is a lot of
subjectivity around what looks "better", so let's fix the issue as
reported and create further discussions as necessary. Then the changes we
make are scoped out and made to all required areas at once, rather than a
series of many small tweaks requiring many reviews. I hope that makes
sense ๐
--
Ticket URL: <
https://code.djangoproject.com/ticket/35404#comment:10>