Skip to content

Conversation

@annavik
Copy link
Member

@annavik annavik commented Oct 23, 2023

Fixes #196

We are basing the page titles on the current breadcrumbs, to be consistent. We just reverse them.

Example of some page titles after this update:

Screenshot 2023-10-23 at 16 58 37

@netlify
Copy link

netlify bot commented Oct 23, 2023

Deploy Preview for ami-web ready!

Name Link
🔨 Latest commit 8ab0f40
🔍 Latest deploy log https://app.netlify.com/sites/ami-web/deploys/65393239d783110008343af8
😎 Deploy Preview https://deploy-preview-288--ami-web.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 51
Accessibility: 100
Best Practices: 92
SEO: 92
PWA: 70
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@annavik
Copy link
Member Author

annavik commented Oct 23, 2023

We could easily be even more detailed here, for example include more information for all details views (not just sessions). Do you think that is a good idea or are titles starting to get too complex? @mihow

@netlify
Copy link

netlify bot commented Oct 23, 2023

Deploy Preview for ami-storybook ready!

Name Link
🔨 Latest commit 8ab0f40
🔍 Latest deploy log https://app.netlify.com/sites/ami-storybook/deploys/6539323981b0560008fac62f
😎 Deploy Preview https://deploy-preview-288--ami-storybook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@mihow
Copy link
Collaborator

mihow commented Oct 25, 2023

Thanks so much @annavik. I do think we should make titles specific to each detail page. Then it will be much easier to navigate through your history and it will be helpful when pasting links in Slack, etc. And it will be much better for search indexes when we go live.

For many detail pages the title could just use the object id "#341 | Occurrences | Vermont ML | AMI Platform". For projects, deployments & species you can use their nice names. Actually even occurrences could be "Actias Luna #241"

@annavik
Copy link
Member Author

annavik commented Oct 25, 2023

Thanks so much @annavik. I do think we should make titles specific to each detail page. Then it will be much easier to navigate through your history and it will be helpful when pasting links in Slack, etc. And it will be much better for search indexes when we go live.

For many detail pages the title could just use the object id "#341 | Occurrences | Vermont ML | AMI Platform". For projects, deployments & species you can use their nice names. Actually even occurrences could be "Actias Luna #241"

Fixed! Added more specific titles for jobs, deployments, occurrences and species. Example:

Screenshot 2023-10-25 at 14 57 58

@annavik
Copy link
Member Author

annavik commented Oct 25, 2023

The update also affects detail breadcrumb, although most of the time it will not be visible behind dialog. If we decide to go from dialog -> full page for more detail views it could be nice to have though.

Screenshot 2023-10-25 at 15 01 02

@annavik
Copy link
Member Author

annavik commented Oct 25, 2023

@mihow I also included a small change to update description when a project is selected. Looks a bit nicer when we share links. From Slack:
Screenshot 2023-10-25 at 17 25 26

See #294

Copy link
Collaborator

@mihow mihow left a comment

Choose a reason for hiding this comment

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

Love it! Thank you for adding the meta previews. We can add images eventually when we get thumbnailing working!

@mihow mihow merged commit e92756a into main Oct 31, 2023
mihow pushed a commit that referenced this pull request Nov 2, 2023
* Add detailed page titles

* Make titles specific to each detail page

* Update description to project description

* Update default description
mihow pushed a commit that referenced this pull request Nov 2, 2023
* Add detailed page titles

* Make titles specific to each detail page

* Update description to project description

* Update default description
@mihow mihow deleted the web-ui-page-titles branch November 17, 2023 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add specific page titles

3 participants