Skip to content

Conversation

@Pytal
Copy link
Member

@Pytal Pytal commented Jan 10, 2023

Before After
image image

Differentiate between share links especially when using a screenreader

@Pytal Pytal added this to the Nextcloud 26 milestone Jan 10, 2023
@Pytal Pytal self-assigned this Jan 10, 2023
@Pytal
Copy link
Member Author

Pytal commented Jan 10, 2023

/backport to stable25

Copy link
Member

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

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

👍

too bad the user cannot give a name to those links

Copy link
Member

@jancborchardt jancborchardt left a comment

Choose a reason for hiding this comment

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

@Pytal the numbers only appear with the second link, right? So "Share link" will initially not have a number. Then I'm thinking it might be best to cut the " (1)" from the first one and only number from then on, just like we do for filenames. when copying files.

@Pytal Pytal force-pushed the enh/a11y-share-link-index branch from c8866bf to 42399fc Compare January 11, 2023 02:09
@Pytal Pytal added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jan 11, 2023
@Pytal Pytal force-pushed the enh/a11y-share-link-index branch from 42399fc to 1334921 Compare January 12, 2023 01:42
@Pytal Pytal force-pushed the enh/a11y-share-link-index branch from 1334921 to 3bdf383 Compare January 12, 2023 17:13
Signed-off-by: Christopher Ng <chrng8@gmail.com>
@Pytal Pytal force-pushed the enh/a11y-share-link-index branch from 3bdf383 to f09300b Compare January 12, 2023 17:24
@Pytal
Copy link
Member Author

Pytal commented Jan 12, 2023

Wording adjusted slightly

@Pytal Pytal merged commit bec52be into master Jan 12, 2023
@Pytal Pytal deleted the enh/a11y-share-link-index branch January 12, 2023 22:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish accessibility feature: sharing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants