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

feat(AppNavigationItem): Allow native title attribute control #3617

Merged
merged 1 commit into from
Jan 10, 2023

Conversation

skjnldsv
Copy link
Contributor

@skjnldsv skjnldsv commented Jan 5, 2023

From nc25 (not vue):
Capture d’écran_2023-01-05_17-10-04

Since we're moving away from tooltips libraries.

<NcAppNavigationItem name="My title" icon="icon-category-enabled" />
<NcAppNavigationItem name="My title" title="My custom title" icon="icon-category-enabled" />

@skjnldsv skjnldsv added 3. to review Waiting for reviews accessibility Making sure we design for the widest range of people possible, including those who have disabilities labels Jan 5, 2023
@skjnldsv skjnldsv self-assigned this Jan 5, 2023
@raimund-schluessler

This comment was marked as resolved.

@skjnldsv

This comment was marked as resolved.

@raimund-schluessler

This comment was marked as resolved.

@skjnldsv

This comment was marked as resolved.

@skjnldsv
Copy link
Contributor Author

skjnldsv commented Jan 5, 2023

Done, please review again @raimund-schluessler

@skjnldsv skjnldsv requested review from a team, Pytal and szaimen and removed request for a team January 6, 2023 08:49
Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
@raimund-schluessler
Copy link
Contributor

Rebased.

@skjnldsv skjnldsv merged commit 83b054e into master Jan 10, 2023
@skjnldsv skjnldsv deleted the fix/title-nav-entry branch January 10, 2023 10:07
@skjnldsv skjnldsv added 4. to release Ready to be released and/or waiting for tests to finish enhancement New feature or request and removed 3. to review Waiting for reviews labels Jan 10, 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 enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants