Skip to content

fix(wordings): Shorter labels for the share action buttons (to avoid longer words being truncated in languages other than English) - #62880

Open
Jerome-Herbinet wants to merge 1 commit into
masterfrom
Jerome-Herbinet-shorter-share-buttons-actions-wordings
Open

fix(wordings): Shorter labels for the share action buttons (to avoid longer words being truncated in languages other than English)#62880
Jerome-Herbinet wants to merge 1 commit into
masterfrom
Jerome-Herbinet-shorter-share-buttons-actions-wordings

Conversation

@Jerome-Herbinet

Copy link
Copy Markdown
Member

⚠️ Important : not tested (no test environment)

Current problem in French (many other languages may have the same issue) :

  • buttons are very large
  • some texts are truncated (ellipsis)
2026-08-04_13-52

I suggest using only action verbs (one word per button) so that the information is clear and easy to read, and so that the interface isn't cluttered with buttons that are too large.

@Jerome-Herbinet
Jerome-Herbinet requested a review from a team as a code owner August 4, 2026 11:56
@Jerome-Herbinet Jerome-Herbinet added design Design, UI, UX, etc. 3. to review Waiting for reviews labels Aug 4, 2026
@Jerome-Herbinet
Jerome-Herbinet requested review from susnux and removed request for a team August 4, 2026 11:56
Comment thread apps/files_sharing/src/views/SharingDetailsTab.vue Outdated
Comment thread apps/files_sharing/src/views/SharingDetailsTab.vue Outdated
@Jerome-Herbinet
Jerome-Herbinet force-pushed the Jerome-Herbinet-shorter-share-buttons-actions-wordings branch 2 times, most recently from 6da47b6 to 9b662d4 Compare August 4, 2026 12:12
@Jerome-Herbinet

Copy link
Copy Markdown
Member Author

@susnux : I have applied you co-authored commits and squashed all of them ; please check.

@Jerome-Herbinet
Jerome-Herbinet force-pushed the Jerome-Herbinet-shorter-share-buttons-actions-wordings branch from 9b662d4 to d63dfcf Compare August 4, 2026 12:13
@Jerome-Herbinet
Jerome-Herbinet requested a review from susnux August 4, 2026 12:13
Comment thread apps/files_sharing/src/views/SharingDetailsTab.vue Outdated
Comment thread apps/files_sharing/src/views/SharingDetailsTab.vue Outdated
Comment thread apps/files_sharing/src/views/SharingDetailsTab.vue Outdated
Comment thread apps/files_sharing/src/views/SharingDetailsTab.vue Outdated
@Jerome-Herbinet
Jerome-Herbinet force-pushed the Jerome-Herbinet-shorter-share-buttons-actions-wordings branch from 38ef559 to c6b3882 Compare August 4, 2026 13:13
@Jerome-Herbinet
Jerome-Herbinet requested a review from susnux August 4, 2026 13:13
Comment thread apps/files_sharing/src/views/SharingDetailsTab.vue Outdated
@Jerome-Herbinet
Jerome-Herbinet force-pushed the Jerome-Herbinet-shorter-share-buttons-actions-wordings branch from 1d231c2 to 73991c1 Compare August 4, 2026 14:16
@Jerome-Herbinet
Jerome-Herbinet requested a review from susnux August 4, 2026 14:17
Comment thread apps/files_sharing/src/views/SharingDetailsTab.vue

@susnux susnux left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Others look great now :)

@Jerome-Herbinet
Jerome-Herbinet force-pushed the Jerome-Herbinet-shorter-share-buttons-actions-wordings branch 2 times, most recently from 7e5a959 to 1696566 Compare August 5, 2026 09:00
@Jerome-Herbinet

Copy link
Copy Markdown
Member Author

Others look great now :)

Done ! :-)

@Jerome-Herbinet
Jerome-Herbinet force-pushed the Jerome-Herbinet-shorter-share-buttons-actions-wordings branch from 1696566 to be12514 Compare August 5, 2026 10:07
@susnux
susnux requested a review from nfebe August 5, 2026 10:10
@Jerome-Herbinet
Jerome-Herbinet force-pushed the Jerome-Herbinet-shorter-share-buttons-actions-wordings branch from be12514 to 23cd771 Compare August 5, 2026 12:49
@Jerome-Herbinet
Jerome-Herbinet force-pushed the Jerome-Herbinet-shorter-share-buttons-actions-wordings branch 3 times, most recently from 8fe5a6d to ebafca7 Compare August 6, 2026 11:48
…tons (to avoid longer words being truncated in languages other than English)

Signed-off-by: Jérôme Herbinet <33763786+Jerome-Herbinet@users.noreply.github.com>

fix(wordings): fix(wordings): Shorter labels for the share action buttons (to avoid longer words being truncated in languages other than English)

Signed-off-by: Jérôme Herbinet <33763786+Jerome-Herbinet@users.noreply.github.com>

fix(wordings): Shorter labels for the share action buttons (to avoid longer words being truncated in languages other than English)

Signed-off-by: Jérôme Herbinet <33763786+Jerome-Herbinet@users.noreply.github.com>

fix(wordings): Shorter labels for the share action buttons (to avoid longer words being truncated in languages other than English)

Signed-off-by: Jérôme Herbinet <33763786+Jerome-Herbinet@users.noreply.github.com>

Update apps/files_sharing/src/views/SharingDetailsTab.vue

Signed-off-by: Jérôme Herbinet <33763786+Jerome-Herbinet@users.noreply.github.com>

fix(comment): adding comments

Signed-off-by: Jérôme Herbinet <33763786+Jerome-Herbinet@users.noreply.github.com>

fix(comment): add one last comment

Signed-off-by: Jérôme Herbinet <33763786+Jerome-Herbinet@users.noreply.github.com>
Co-Authored-By: Ferdinand Thiessen <opensource@fthiessen.de>
@Jerome-Herbinet
Jerome-Herbinet force-pushed the Jerome-Herbinet-shorter-share-buttons-actions-wordings branch from ebafca7 to 8310c78 Compare August 7, 2026 12:00
@susnux

susnux commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

playwright tests are related as the labels have been changed the POM has to be changed

@Jerome-Herbinet

Copy link
Copy Markdown
Member Author

playwright tests are related as the labels have been changed the POM has to be changed

I'm sorry, @susnux … I'm a little confused because I don't understand what I'm supposed to do (I lack expertise in this particular area). Could you give me some guidance?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews design Design, UI, UX, etc. feature: sharing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants