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

Add 3D/2D view #465

Merged
merged 1 commit into from
Jul 1, 2024
Merged

Add 3D/2D view #465

merged 1 commit into from
Jul 1, 2024

Conversation

whunter
Copy link
Member

@whunter whunter commented Jun 4, 2024

Adds view to render x3dom model and mirador viewer in same display and allow toggling between them.


JIRA Ticket: (link) (:star:)

  • Other Relevant Links (Meeting note, project page, related pull requests, etc.)

What does this Pull Request do? (:star:)

Adds view to render x3dom model and mirador viewer in same display and allow toggling between them.

What's the changes? (:star:)

  • Adds new view option that contains:
    • x3dom
    • mirador
    • leaflet thumbnail w/ location displayed

How should this be tested?

  • View 3d/2d record here: https://3d-view-refactor.d2ysrrdhih4bgc.amplifyapp.com/archive/j636089g
  • Check that 3d model loads
  • Check that leaflet thumbnail loads with correct content.
  • Check that you can click "View 2D Full Size" link and it will toggle to Mirador view
  • Check that you can toggle back and forth between views
  • Check other records to make sure that iiif only records (no 3d) are still functional

Already imported 3D/2D records include:
VTEC000004835
VTEC000004836
VTEC000004837
VTEC000004838
VTEC000004841
VTEC000004842
VTEC000004843
VTEC000004844
VTEC000004845
VTEC000004847
VTEC000004848
VTEC000004849
VTEC000004850
VTEC000004851
VTEC000004853
VTEC000004854

Additional Notes:

  • branch: 3d_view_refactor

Interested parties

@goynejennifer @otokama

(:star:) Required fields

mostly working?

add map w/ styles

comment--

toggle visibility. load 3d larger

fix mirador

it's reloading for now. It's fine

controls

resizing to fit. yay

still working on scaling

move leaflet css

tests and pdfs and such
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-465.d234tdmg9icdwk.amplifyapp.com

@goynejennifer goynejennifer requested review from asthad1 and removed request for goynejennifer June 5, 2024 18:19
@goynejennifer
Copy link
Collaborator

Style change: Can we make the 3D view the entire window 1016px by 720px will cover the whole window
BugsFullView

@asthad1
Copy link
Collaborator

asthad1 commented Jun 6, 2024

The other thing when we click full screen and it appears on full screen the image is kinda grayed out. Also I can no longer use my mouse trackpad to zoom in or zoom out. We have to use the zoom in/out button.
image

@asthad1 asthad1 added enhancement New feature or request and removed Ready for review labels Jun 7, 2024
@whunter whunter merged commit b0ef9bd into dev Jul 1, 2024
1 of 2 checks passed
@whunter whunter deleted the 3d_view_refactor branch July 1, 2024 16:32
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.

3 participants