Skip to content

Updated about links#1329

Merged
mialondon merged 1 commit intoUniversalViewer:devfrom
Geoffsc:1309-improve-about-link
Mar 10, 2025
Merged

Updated about links#1329
mialondon merged 1 commit intoUniversalViewer:devfrom
Geoffsc:1309-improve-about-link

Conversation

@Geoffsc
Copy link
Copy Markdown
Contributor

@Geoffsc Geoffsc commented Mar 7, 2025

#1309

  • Updated "More info" links to point to new "About" wiki page
  • Fixed small config url bug

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 7, 2025

The latest updates on your projects. Learn more about Vercel for Git β†—οΈŽ

Name Status Preview Comments Updated (UTC)
universalviewer βœ… Ready (Inspect) Visit Preview πŸ’¬ Add feedback Mar 7, 2025 1:32pm

Copy link
Copy Markdown
Contributor

@demiankatz demiankatz 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 -- thanks, @Geoffsc!

@LanieOkorodudu
Copy link
Copy Markdown
Collaborator

Thanks for the fix, @Geoffsc! I have tested it, and it works when UV is open on a desktop.

@demiankatz, should the settings button also be available in mobile view to access the "About" functionality?

@demiankatz
Copy link
Copy Markdown
Contributor

@LanieOkorodudu, you raise a good point about the settings dialogue being inaccessible on mobile. I think we should probably open an issue to discuss that situation further, as I suspect it raises some new questions and design considerations. I don't think it needs to delay the completion of this PR, though, as I think that's a separate matter.

@mialondon mialondon merged commit d900349 into UniversalViewer:dev Mar 10, 2025
5 checks passed
@github-project-automation github-project-automation bot moved this from IN TESTING to COMPLETED in Community Sprint Feb 2025 Mar 10, 2025
This was referenced Apr 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Status: COMPLETED

Development

Successfully merging this pull request may close these issues.

4 participants