Sitelet https://web.archive.org/web/20211209020748/https://github.com/zulip/zulip/issues/20077
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Resize "Add choice" poll button based on language #20077

Open
alya opened this issue Oct 27, 2021 · 14 comments · May be fixed by #20090
Open

Resize "Add choice" poll button based on language #20077

alya opened this issue Oct 27, 2021 · 14 comments · May be fixed by #20090

Comments

@alya
Copy link
Collaborator

@alya alya commented Oct 27, 2021

At present, the "Add choice" poll button ends up taking two lines in languages where the text on the button is relatively long. E.g. in Russian:

Screen Shot 2021-10-27 at 11 32 35 AM

Instead, the button should be sized to fit the full text on one line regardless of language. Note that it should still be sized appropriately in languages where the text is short, without a lot of extra space inside the button.

@zulipbot
Copy link
Member

@zulipbot zulipbot commented Oct 27, 2021

Hello @zulip/server-i18n members, this issue was labeled with the "area: i18n" label, so you may want to check it out!

Loading

@gopik820
Copy link
Collaborator

@gopik820 gopik820 commented Oct 27, 2021

@zulipbot claim

Loading

@zulipbot
Copy link
Member

@zulipbot zulipbot commented Oct 27, 2021

Welcome to Zulip, @gopik820! We just sent you an invite to collaborate on this repository at https://github.com/zulip/zulip/invitations. Please accept this invite in order to claim this issue and begin a fun, rewarding experience contributing to Zulip!

Here's some tips to get you off to a good start:

As you work on this issue, you'll also want to refer to the Zulip code contribution guide, as well as the rest of the developer documentation on that site.

See you on the other side (that is, the pull request side)!

Loading

@gopik820 gopik820 linked a pull request that will close this issue Oct 28, 2021
gopik820 added a commit to gopik820/zulip that referenced this issue Oct 28, 2021
gopik820 added a commit to gopik820/zulip that referenced this issue Oct 29, 2021
@alya
Copy link
Collaborator Author

@alya alya commented Oct 30, 2021

Turns out we have the same issue with the "Add task" button in TODO lists:

Screen Shot 2021-10-29 at 10 59 24 PM

Loading

@alya
Copy link
Collaborator Author

@alya alya commented Oct 30, 2021

Also, while we're here, we should check whether we forgot to internationalize the "Task list" string.

Loading

@gopik820
Copy link
Collaborator

@gopik820 gopik820 commented Oct 30, 2021

Turns out we have the same issue with the "Add task" button in TODO lists:

That was also fixed in the PR ,since the width property was common to both buttons.

Loading

@gopik820
Copy link
Collaborator

@gopik820 gopik820 commented Oct 30, 2021

Also, while we're here, we should check whether we forgot to internationalize the "Task list" string.

Yes it seems that the "Task list" string is not internationalized.

Loading

@gopik820
Copy link
Collaborator

@gopik820 gopik820 commented Nov 3, 2021

@alya The PR is ready for review.

Loading

@alya
Copy link
Collaborator Author

@alya alya commented Nov 3, 2021

Cool, could you also fix the "Task list" internationalization? I don't see it in the PR.

Loading

@zulipbot
Copy link
Member

@zulipbot zulipbot commented Nov 14, 2021 •

Hello @gopik820, you have been unassigned from this issue because you have not updated this issue or any referenced pull requests for over 14 days.

You can reclaim this issue or claim any other issue by commenting @zulipbot claim on that issue.

Thanks for your contributions, and hope to see you again soon!

Loading

@LDeolindo
Copy link
Collaborator

@LDeolindo LDeolindo commented Nov 24, 2021

@zulipbot claim

Loading

@zulipbot
Copy link
Member

@zulipbot zulipbot commented Nov 24, 2021

Welcome to Zulip, @LDeolindo! We just sent you an invite to collaborate on this repository at https://github.com/zulip/zulip/invitations. Please accept this invite in order to claim this issue and begin a fun, rewarding experience contributing to Zulip!

Here's some tips to get you off to a good start:

As you work on this issue, you'll also want to refer to the Zulip code contribution guide, as well as the rest of the developer documentation on that site.

See you on the other side (that is, the pull request side)!

Loading

@zulipbot
Copy link
Member

@zulipbot zulipbot commented Dec 4, 2021 •

Hello @LDeolindo, you have been unassigned from this issue because you have not updated this issue or any referenced pull requests for over 14 days.

You can reclaim this issue or claim any other issue by commenting @zulipbot claim on that issue.

Thanks for your contributions, and hope to see you again soon!

Loading

LDeolindo added a commit to LDeolindo/zulip that referenced this issue Dec 4, 2021
LDeolindo added a commit to LDeolindo/zulip that referenced this issue Dec 4, 2021
This issue has been partially fixed by PR zulip#20090.
I continued from it to ensure that the "Add task" button
had been fixed, that the buttons looked good visually,
I just added some paddings to ensure that the buttons stayed
at the same level as the inputs and the text was spaced out.

Tested on my local using manually and visually test.

Fixes zulip#20077
@LDeolindo
Copy link
Collaborator

@LDeolindo LDeolindo commented Dec 9, 2021

@zulipbot claim

Loading

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

4 participants