Opened 10 years ago
Closed 9 years ago
#32111 closed defect (bug) (duplicate)
Problem in spanish version with buttons for publish
Reported by: | ramonjosegn | Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | normal | Version: | 4.2 |
Component: | Posts, Post Types | Keywords: | good-first-bug needs-testing has-patch |
Focuses: | ui, administration | Cc: |
Attachments (2)
Change History (20)
#1
@
10 years ago
- Component changed from General to Posts, Post Types
- Focuses ui administration added
#4
@
9 years ago
- Keywords good-first-bug needs-patch added
- Milestone changed from Awaiting Review to 4.4
#5
follow-up:
↓ 6
@
9 years ago
I managed to fix this issue using the following code. But, I'm sure this can't be that easy and I'm leaving edge cases out.
#save-action { float: left; width: 50%; } #preview-action { float: right; width: 50%; }
#6
in reply to:
↑ 5
@
9 years ago
The problem with that code is that it'll take the spinner to a new line. Plus, for some other languages it will make both buttons overlap.
#8
follow-ups:
↓ 9
↓ 10
@
9 years ago
- Keywords needs-patch added; has-patch removed
@Rynald0s, thanks for the patch! However, minified files should not be patched, our build tools take care of that.
Could you provide a patch for the non-minified file? Use the SCRIPT_DEBUG constant for that.
#9
in reply to:
↑ 8
@
9 years ago
Replying to SergeyBiryukov:
@Rynald0s, thanks for the patch! However, minified files should not be patched, our build tools take care of that.
Could you provide a patch for the non-minified file? Use the SCRIPT_DEBUG constant for that.
Absolutely. Thanks so much, Sergey.
#10
in reply to:
↑ 8
@
9 years ago
- Keywords has-patch added; needs-patch removed
Replying to SergeyBiryukov:
@Rynald0s, thanks for the patch! However, minified files should not be patched, our build tools take care of that.
Could you provide a patch for the non-minified file? Use the SCRIPT_DEBUG constant for that.
done
#11
follow-up:
↓ 12
@
9 years ago
Nice! unfortunately that patch will leave the preview button unaligned with the "Publish" button.
At the same time, for other languages like the ones mentioned in #20842 (I guess russian and arabic), this patch won't take effect, because they have larger strings.
I think the only way to get this solved is thinking about a new distribution for the buttons. With things as they are right now, we'll always depend on the length of the string, and that varies from language to language.
#12
in reply to:
↑ 11
;
follow-up:
↓ 17
@
9 years ago
Replying to juanfra:
Nice! unfortunately that patch will leave the preview button unaligned with the "Publish" button.
At the same time, for other languages like the ones mentioned in #20842 (I guess russian and arabic), this patch won't take effect, because they have larger strings.
I think the only way to get this solved is thinking about a new distribution for the buttons. With things as they are right now, we'll always depend on the length of the string, and that varies from language to language.
Was looking at the translated strings for Russian and Arabic languages (#20842). The string in Russian language you see in the image doesn't correspond with the string in the translation file as it is today. Then again, this was 3 years ago so it appears that this issue was resolved by simply taking the languages that has long strings ('Save Draft' button), and making them smaller.
That said, disregard the proposed patch and instead, we could just change the string in Spanish language files from "Guardar borrador", to only "Guardar", and in Catalan language files from "Desa l'esborrany" to only "Desa." The same has been done for Russian language. This will be the new proposed 'patch'.
I have already made the translation suggestions for Spanish & Catalan languages. As per #20842, both Russian and Arabic displays correctly so this could be marked as complete/solved.
Thanks
#13
follow-up:
↓ 14
@
9 years ago
Awesome. Have you sent a message with this new translation suggestion?
#14
in reply to:
↑ 13
@
9 years ago
Replying to juanfra:
Awesome. Have you sent a message with this new translation suggestion?
I have suggested the new translation strings.
#15
follow-up:
↓ 16
@
9 years ago
Yep, I saw the new string suggestion in polyglots. But, have you accompanied that suggestion with a message to the reviewer with a reference to this ticket?
I've contacted the Spanish translation team in order to aware them of this new suggestion.
#16
in reply to:
↑ 15
@
9 years ago
Replying to juanfra:
Yep, I saw the new string suggestion in polyglots. But, have you accompanied that suggestion with a message to the reviewer with a reference to this ticket?
I've contacted the Spanish translation team in order to aware them of this new suggestion.
Ah, no I have not done that. Thanks for letting them know.
#17
in reply to:
↑ 12
@
9 years ago
Replying to Rynald0s:
Was looking at the translated strings for Russian and Arabic languages (#20842). The string in Russian language you see in the image doesn't correspond with the string in the translation file as it is today. Then again, this was 3 years ago so it appears that this issue was resolved by simply taking the languages that has long strings ('Save Draft' button), and making them smaller.
Actually, the string is still there:
And the screenshot from comment:4:ticket:20842 is still relevant. The string in question is not "Save Draft", but "Save as Pending", which is displayed for users with Contributor role.
The same is happening in the Catalan version