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(NcAppContent): Allow to set the page title #5269

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

susnux
Copy link
Contributor

@susnux susnux commented Feb 20, 2024

☑️ Resolves

Sometimes - e.g. when you have multiple view - you want to adjust the page title, that is shown as the browsers tab name. This adds a property to set that value.

🏁 Checklist

  • ⛑️ Tests are included or are not applicable
  • 📘 Component documentation has been extended, updated or is not applicable
  • 3️⃣ Backport to next requested with a Vue 3 upgrade

@susnux susnux added 3. to review Waiting for reviews feature: app-content Related to the app-content component labels Feb 20, 2024
@susnux susnux added this to the 8.8.0 milestone Feb 20, 2024
src/components/NcAppContent/NcAppContent.vue Outdated Show resolved Hide resolved
src/components/NcAppContent/NcAppContent.vue Outdated Show resolved Hide resolved
src/components/NcAppContent/NcAppContent.vue Outdated Show resolved Hide resolved
@susnux susnux force-pushed the feat/allow-setting-page-title branch from aa0f4bf to 1181ddb Compare February 20, 2024 22:14
@susnux susnux force-pushed the feat/allow-setting-page-title branch from 5f8b7d9 to c5f8dac Compare February 29, 2024 13:26
@susnux susnux added the enhancement New feature or request label Feb 29, 2024
@susnux susnux force-pushed the feat/allow-setting-page-title branch from c5f8dac to d75d10a Compare February 29, 2024 13:32
@Antreesy Antreesy modified the milestones: 8.8.0, 8.8.2 Feb 29, 2024
@susnux susnux modified the milestones: 8.8.1, 8.9.0 Feb 29, 2024
Copy link
Contributor

@ShGKme ShGKme left a comment

Choose a reason for hiding this comment

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

Everything looks fine, cool feature.

Only one old thing with the translations

src/components/NcAppContent/NcAppContent.vue Outdated Show resolved Hide resolved
@juliushaertl juliushaertl modified the milestones: 8.9.0, 8.10.0 Mar 6, 2024
@ShGKme ShGKme modified the milestones: 8.10.0, 8.11.0, 8.11.1 Mar 11, 2024
@ShGKme ShGKme added this to the 8.12.0 milestone Mar 15, 2024
susnux and others added 2 commits May 10, 2024 20:04
Sometimes - e.g. when you have multiple view - you want to adjust the page title,
that is shown as the browsers tab name. This adds a property to set that value.

Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de>
Co-authored-by: Grigorii K. Shartsev <me@shgk.me>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux susnux force-pushed the feat/allow-setting-page-title branch from d75d10a to 62b6d86 Compare May 10, 2024 18:49
@susnux susnux requested a review from ShGKme May 10, 2024 21:03
@susnux susnux modified the milestones: 8.12.0, 8.13.0 May 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews enhancement New feature or request feature: app-content Related to the app-content component
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants