secondlife / secondlife/viewer

Update GLTF Model Upload window to include uploaded materials

Open
#4,131 6 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement triage
Dominant language
C++
Stars
299
Forks
146
Avg merge
1d 9h
Merged PRs (30d)
88

Description

Currently uploading a GLTF mesh (that has applied materials) gives both an 'Upload Model' floater and an 'Upload Material' floater.
This can appear messy and confusing.
Image

An updated GLTF model floater should ideally include both the mesh and material information.

Depending on the viewers window size, it is also possible for a material upload dialog to be completely hidden as shown in https://github.com/secondlife/viewer/issues/4108

Test Plan

  • Upload a GLTF model with a single material
  • Upload a GLTF model with more than one material

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Reproduce the two test cases by uploading GLTF models with one and multiple materials, then inspect the existing Upload Model and Upload Material floaters and their interaction. Done means the model floater presents both mesh and material information without a separate confusing or hidden material dialog.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
computer-graphics, desktop
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.