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

CORE-1968 Add page titles in each DE page #552

Merged
merged 2 commits into from
Oct 27, 2023

Conversation

psarando
Copy link
Member

This PR will add a basic page title to each DE page.

For example, the home page will still be titled Discovery Environment, but this PR will update the page title for the /data page to Data - Discovery Environment, the /apps listing page to Apps - Discovery Environment, the /help page to Help - Discovery Environment, etc.

Some pages will add additional info to the page title, if available.

For example, when viewing the details of a collection or team, the collection or team name will also display in the title, as Collection - name - Discovery Environment or Team - name - Discovery Environment; and when a user navigates to their home folder under /data/ds/cyverse/home/user, then the page title will display as Data - /cyverse/home/user - Discovery Environment.

When viewing subfolders of a user's home folder, then only the name of the current file or folder will display in the title (e.g. Data - current_folder - Discovery Environment).

App categories will also display in the page title when viewing the different app listings, but because the category names usually also contain the word "App", then the title will not have App - prepended to it (e.g. just Favorite Apps - Discovery Environment).

Set the path/file/folder in the page title for the data listing page,
app category name in the app listing page,
team/collection name in the teams/collections pages,
and the search term in the subscriptions and search pages.
@psarando psarando added the enhancement New feature or request label Oct 26, 2023
@psarando psarando requested review from slr71 and sboleyn October 26, 2023 23:10
Copy link
Member

@slr71 slr71 left a comment

Choose a reason for hiding this comment

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

Looks good to me! 👍 This will be a great feature to have. 🎉

@slr71
Copy link
Member

slr71 commented Oct 27, 2023

I almost added a 🌮 emoji by accident. I guess it would have been applicable. Tacos are good to have too. 🤣

@psarando
Copy link
Member Author

I almost added a 🌮 emoji by accident. I guess it would have been applicable. Tacos are good to have too. 🤣

💯 😋

Thanks for the review! 👍

@psarando psarando merged commit ceda13e into cyverse-de:master Oct 27, 2023
2 checks passed
@psarando psarando deleted the CORE-1968-page-titles branch October 27, 2023 01:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants