Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#28885 closed Bug (fixed)

Bottom parts of "Congratulations!" page cut off internationalized content

Reported by: Maciej Olko Owned by: Himanshu Chauhan
Component: Core (Other) Version: 2.0
Severity: Normal Keywords: i18n, template, defaults
Cc: Timothy Allen Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: yes

Description (last modified by Maciej Olko)

Bottom parts (footer) of "Congratulations!" page[1] (rocket taking off/default urlconf) cut off text if it is too long. It happens for e.g. Russian and Lower Serbian languages. Screenshots:
https://screenshots.firefox.com/H4OzOchrVQCvNp5e/localhost
https://screenshots.firefox.com/dG8BTsWHB4s0F2pL/localhost

For many other languages, the problem is more of just esthetic nature, as the padding to the bottom of page is not equal to padding to the footer top ruler (if translations aren't one-liners in effect).

The footer shouldn't have fixed height. I am afraid that some JavaScript is neccessary to calculate footer's height if we are about to keep the layout.

[1] django/django/views/templates/default_urlconf.html

Change History (11)

comment:1 by Claude Paroz, 7 years ago

Cc: Timothy Allen added
Triage Stage: UnreviewedAccepted

comment:2 by Timothy Allen, 7 years ago

Owner: changed from nobody to Timothy Allen
Status: newassigned

We will tweak the design so it works in more languages, and test with Google Translate. Thanks for the catch!

comment:3 by Maciej Olko, 7 years ago

Description: modified (diff)

You can also test it with official Django translations, by just changing the LANGUAGE_CODE setting. Language codes that have longest translations (AFAIK) are "ru" and "dsb".

comment:4 by Himanshu Chauhan, 7 years ago

I have tweaked the CSS of the web page so that it does not change original design and fix the mentioned bug. Can I claim this ticket and submit the patch?

in reply to:  4 comment:5 by Timothy Allen, 7 years ago

Replying to Himanshu Chauhan:

I have tweaked the CSS of the web page so that it does not change original design and fix the mentioned bug. Can I claim this ticket and submit the patch?

Absolutely! My colleague and I were working on a fix, but if you've got something complete, by all means take the ticket and issue a P.R. Thanks so much!

comment:6 by Timothy Allen, 7 years ago

Owner: Timothy Allen removed
Status: assignednew

comment:7 by Himanshu Chauhan, 7 years ago

Owner: set to Himanshu Chauhan
Status: newassigned

comment:8 by Himanshu Chauhan, 7 years ago

Has patch: set

comment:9 by Carlton Gibson, 7 years ago

Triage Stage: AcceptedReady for checkin

The patch looks good. (Screenshots on PR)

The commit message does not quite match the guidelines, but could be adjusted when merging.

Need to confirm Himanshu submitted the CLA.

Update: both these queries addressed in PR https://github.com/django/django/pull/9570#issuecomment-356871766

Last edited 7 years ago by Carlton Gibson (previous) (diff)

comment:10 by Tim Graham <timograham@…>, 7 years ago

Resolution: fixed
Status: assignedclosed

In 1b753b2:

Fixed #28885 -- Fixed hidden content at the bottom of the "The install worked successfully!" page for some languages.

comment:11 by Tim Graham <timograham@…>, 7 years ago

In cc1bdce6:

[2.0.x] Fixed #28885 -- Fixed hidden content at the bottom of the "The install worked successfully!" page for some languages.

Backport of 1b753b2d60230974e7d537ddacbcc1ad23348e7c from master

Note: See TracTickets for help on using tickets.
Back to Top