Skip to content

fix(NcAppSettingsDialog): respect showNavigation prop#4954

Merged
Pytal merged 2 commits intomasterfrom
fix/4941/NcAppSettingsDialog--respect-show-navigation-prop
Dec 14, 2023
Merged

fix(NcAppSettingsDialog): respect showNavigation prop#4954
Pytal merged 2 commits intomasterfrom
fix/4941/NcAppSettingsDialog--respect-show-navigation-prop

Conversation

@ShGKme
Copy link
Contributor

@ShGKme ShGKme commented Dec 12, 2023

☑️ Resolves

🖼️ Screenshots

With :show-navigation="false"

🏚️ Before 🏡 After
image image

🏁 Checklist

  • ⛑️ Tests are included or are not applicable
  • 📘 Component documentation has been extended, updated or is not applicable

Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
@ShGKme ShGKme added bug Something isn't working 3. to review Waiting for reviews feature: settings Related to the settings component labels Dec 12, 2023
@ShGKme ShGKme added this to the 8.3.1 milestone Dec 12, 2023
@ShGKme ShGKme self-assigned this Dec 12, 2023
Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
@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 Dec 14, 2023
@Pytal Pytal merged commit d0a4f5b into master Dec 14, 2023
@Pytal Pytal deleted the fix/4941/NcAppSettingsDialog--respect-show-navigation-prop branch December 14, 2023 20:25
@ShGKme ShGKme mentioned this pull request Dec 22, 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 bug Something isn't working feature: settings Related to the settings component

Projects

None yet

Development

Successfully merging this pull request may close these issues.

NcAppSettingsDialog :show-navigation="false" does not hide the navigation

3 participants