Skip to content

Comments

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

Merged
skjnldsv merged 1 commit intomasterfrom
fix/title-nav-entry
Jan 10, 2023
Merged

feat(AppNavigationItem): Allow native title attribute control#3617
skjnldsv merged 1 commit intomasterfrom
fix/title-nav-entry

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 skjnldsv force-pushed the fix/title-nav-entry branch from beff6ad to a5a2f87 Compare January 5, 2023 17:35
@skjnldsv
Copy link
Contributor Author

skjnldsv commented Jan 5, 2023

Done, please review again @raimund-schluessler

@skjnldsv skjnldsv force-pushed the fix/title-nav-entry branch 2 times, most recently from a8a0ffa to 5936a88 Compare January 5, 2023 17:47
@skjnldsv skjnldsv force-pushed the fix/title-nav-entry branch from 5936a88 to 5f2ff98 Compare January 5, 2023 17:51
@skjnldsv skjnldsv force-pushed the fix/title-nav-entry branch from 5f2ff98 to 47cffb4 Compare January 6, 2023 08:40
@skjnldsv skjnldsv force-pushed the fix/title-nav-entry branch from 47cffb4 to 33aeadf Compare January 6, 2023 08:47
@skjnldsv skjnldsv requested review from a team, Pytal and szaimen and removed request for a team January 6, 2023 08:49
@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