Richie
(Richie Rich)
2024 年9 月 8 日 11:02
1
Hi everyone.
Just updated to Disourse 3.4.0.beta2-dev (cde4db1c6a ) only to find the layout of the admin page for Badges is a bit broken.
The list of badges is usually on the left of the screen, allowing you to manage / add / edit the badges on the right hand side of the screen.
But here’s how it looks now, there’s a great big empty and unusable space in the left-middle:
1 个赞
Lilly
(Lillian Louis)
2024 年9 月 8 日 12:27
2
can you try safe mode ? I can’t reproduce this.
1 个赞
Richie
(Richie Rich)
2024 年9 月 8 日 12:52
3
Hmm, it’s doing it in Safari and Chrome, both tested and replicated in Safe Mode too.
Ok, thanks @Lilly - it might be something else I’ve changed elsewhere, you can probably ignore this one
1 个赞
Lilly
(Lillian Louis)
2024 年9 月 8 日 12:54
4
I just did a fresh deploy to my test hosted site and badge admin is working as expected. that is odd that it’s doing that in safe mode .
I also did a check with header drop down menu instead of sidebar, as well as using the old admin nav menu, and it is still working as expected.
4 个赞
Richie
(Richie Rich)
2024 年9 月 9 日 07:54
5
Steps to replicate:
Change the default locale
setting to English (UK)
:
Reload Discourse in a new tab and browse to https://example.com/admin/badges
where the layout is now broken:
Change or reset the default locale
setting to it’s default of English (US)
:
Reload Discourse in a new tab and browse to https://example.com/admin/badges
where the layout is now working just fine:
I have absolutely no idea how or why this is the case
But a big shout out to @JammyDodger for highlighting the root cause
Is this bug something that could be fixed?
4 个赞
david
(David Taylor)
2024 年9 月 9 日 11:53
6
Thanks @Richie !
Looks like there are a couple of problems here.
The English(UK) translation is missing the new, longer, description at the top of the page. That’s because the ‘badges.description’ key has been reused, and we haven’t done an update from Crowdin since then
This will resolve itself imminently, when translations are auto-updated
The layout seems to be dependent on the length of the description. In US english, I can repro the problem by going into dev tools and shortening the description
Even once the UK english string is updated, this is still worth fixing because other languages which use fewer characters are likely to have the same problem.
Looks like the change is related to UX: Apply admin UI to Badges (#28724) · discourse/discourse@be5c37a · GitHub (cc @ella @martin )
6 个赞
Richie
(Richie Rich)
2024 年9 月 9 日 12:29
8
Thanks for the insight @david
4 个赞
Lilly
(Lillian Louis)
2024 年9 月 9 日 14:33
9
I’m in Canada and I’m blaming the UK for this one.
thanks Richie. I didn’t even consider the locale setting
5 个赞
Jagster
(Jakke Lehtonen)
2024 年9 月 9 日 15:52
10
This is totally off topic, but there isn’t any English (CA), there is? So would you use UK or US locales? It may depend of course, but in general?
3 个赞
Lilly
(Lillian Louis)
2024 年9 月 9 日 15:53
11
our stuff usually defaults to US. we don’t say loo or trousers much here. lol
4 个赞
Richie
(Richie Rich)
2024 年9 月 9 日 19:17
12
Not a problem, we’ll happily claim ownership of this one on behalf of the rest of the world
4 个赞
ella
(Ella)
2024 年9 月 9 日 19:46
13
Thanks for catching that!
I added a div wrapper to make the layout more stable and fixed the issue with inconsistency when using different locales.
Should be fixed with this PR:
discourse:main
← discourse:fix-admin-badge-layout
opened 05:44PM - 09 Sep 24 UTC
- Added a div wrapper around the content below the description to stabilize the … layout
- Fixed an issue where using a locale other than US English caused layout inconsistencies
Reported here: https://meta.discourse.org/t/layout-of-admin-badges-page-broken-after-updating/325492
6 个赞
Richie
(Richie Rich)
2024 年9 月 10 日 12:24
14
My entire country thanks you for the quick fix @ella
3 个赞