Skip to content

enh(NcActions): Navigation link actions - #4084

Merged
Pytal merged 1 commit into
masterfrom
enh/a11y-nav-actions
May 25, 2023
Merged

enh(NcActions): Navigation link actions#4084
Pytal merged 1 commit into
masterfrom
enh/a11y-nav-actions

Conversation

@Pytal

@Pytal Pytal commented May 11, 2023

Copy link
Copy Markdown
Contributor

@Pytal

Pytal commented May 11, 2023

Copy link
Copy Markdown
Contributor Author

/backport to stable7

Comment thread src/components/NcActions/NcActions.vue Outdated
@Pytal
Pytal force-pushed the enh/a11y-nav-actions branch from 4bbf94e to 1da987e Compare May 12, 2023 01:23
@Pytal
Pytal requested a review from skjnldsv May 12, 2023 01:31
@skjnldsv

Copy link
Copy Markdown
Contributor

I haven't realized that would look that complex 🙈
I can approve, but I'm really unsure

@Pytal

Pytal commented May 12, 2023

Copy link
Copy Markdown
Contributor Author

I haven't realized that would look that complex 🙈 I can approve, but I'm really unsure

Either way is fine with me, just the initial reason for not detecting it automatically in the first place was that the overhead didn't seem worth it

Let me know which way you'd think is best and I'll update as needed :)

@Pytal

Pytal commented May 24, 2023

Copy link
Copy Markdown
Contributor Author

Which one do you think is best @skjnldsv?

@skjnldsv

Copy link
Copy Markdown
Contributor

Which one do you think is best @skjnldsv?

I don't know.
I feel both solutions have their own dilemmas :/

@artonge artonge 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.

Auto-detecting looks ok to me.
I would add a comment, though.

Signed-off-by: Christopher Ng <chrng8@gmail.com>
@Pytal
Pytal force-pushed the enh/a11y-nav-actions branch from 1da987e to ff5ad4b Compare May 25, 2023 16:43
@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 May 25, 2023
@Pytal
Pytal merged commit e11cc33 into master May 25, 2023
@Pytal
Pytal deleted the enh/a11y-nav-actions branch May 25, 2023 17:40
@Pytal Pytal mentioned this pull request Jun 6, 2023
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 Making sure we design for the widest range of people possible, including those who have disabilities feature: actions Related to the actions components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants