Skip to content

Updated about links#1329

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

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

Conversation

@Geoffsc

@Geoffsc Geoffsc commented Mar 7, 2025

Copy link
Copy Markdown
Contributor

#1309

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

@vercel

vercel Bot commented Mar 7, 2025

Copy link
Copy Markdown

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

@demiankatz demiankatz left a comment

Copy link
Copy Markdown
Contributor

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
Contributor

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
@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