Skip to content

3D Viewer UX enhancement

Fixes

Improves 3D viewer by @msquare-2

Description

Type of PR This PR is an improvement.

Links

3D Viewer home page: http://localhost:2354/3dviewer

3D Viewer for model P215518: http://localhost:2354/3dviewer/P215518

3D Viewer for model P360985: http://localhost:2354/3dviewer/P360985

Tests

1)Please insert table images into cdli_db by importing images__1_.sql or add the following record manually:

  • id = auto, artifact_id = 215518, is_public = 1

  • id = auto, artifact_id = 360985, is_public = 1

2)Please add below entry to postings table in cdli_db:

  • id=193, posting_type_id=2, slug=threed-viewer-info, artifact_id=0, Published=1, title=CDLI 3D Viewer and 3D Models, body=body_text.html, lang=eng.

3)Check Shortcut keys for:

  • Move---->"M"

  • Rotate---->" R "

  • Zoom In---->" + "

  • Zoom Out---->" - "

  • Undu---->" Z "

  • Lighting---->" L "

  • Fullscreen---->" F "

  • Unfullscreen---->" Esc "

4)Check Damping effect for tablet interaction(Damping--> Animation effect with model movement).

5)Check for all icons are updated as of in the below screenshot.

6)Access barrier can be checked by changing the following permission in db for the above-mentioned models(Currently we have both models public).

  • is_public in the images table.

  • are_images_public in the artifacts table.

  • is_public in the artifacts table.

Note: To access private models, need to log in as admin.

Screenshots

Home Page 3dviewer_home_page

3D Viewer 3dviewer_without_envelope

3D Viewer for Envelope 3dviewer_with_envelope

Checklist:

  • My pull request has a descriptive title (not a vague title like "Update index.md").
  • My pull request targets the phoenix/develop branch of the repository.
  • My commit messages follow best practices.
  • My code follows the established code style of the repository.
  • I added tests for the changes I made (if applicable).
  • I added or updated documentation (if applicable).
  • I tried running the project locally and verified that there are no visible errors.
Edited by Mustafa Dhar

Merge request reports

Loading