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

LOD Slider for BSLODTriShape visualization #22

Closed
3 of 4 tasks
hexabits opened this issue Feb 4, 2014 · 3 comments
Closed
3 of 4 tasks

LOD Slider for BSLODTriShape visualization #22

hexabits opened this issue Feb 4, 2014 · 3 comments

Comments

@hexabits
Copy link
Member

hexabits commented Feb 4, 2014

Info: http://niftools.sourceforge.net/forum/viewtopic.php?f=4&t=3363&start=0#p23326

Task List

  • UI/widget for manipulating LOD Level
  • Show triangles for LOD Level set in widget. 2 = {0, 1, 2} (all visible), 1 = {0, 1}
  • Hide/Disable slider for meshes without BSLODTriShapes
  • Usability improvements to current implementation
@hexabits hexabits added this to the v2.0.0 milestone Mar 12, 2014
@hexabits hexabits self-assigned this Mar 12, 2014
hexabits referenced this issue in hexabits/nifskope May 14, 2014
@hexabits
Copy link
Member Author

Implementation is currently a bit rudimentary... can improve the UI, but the underlying logic seems sound and shouldn't need changing.

Example:

meshes/architecture/whiterun/wrbuildings/wrjorvaskr01.nif

Level 1
level1
Level 2 (Everything visible)
level2

(There is no difference between Level 0 and Level 1 on this mesh)

Slider position is saved with other app settings and thus is maintained across sessions. Given the sparse UI currently, this may lead to confusion. Maybe it should always default to 2...

@hexabits
Copy link
Member Author

Updated tasks to include:

  • Hide slider for meshes without BSLODTriShapes

It will be a small amount of work to do so. Alternative being to at least show it for just the appropriate user versions.

@hexabits
Copy link
Member Author

Current Issue: I originally implemented the enable/disable for the slider with signals but it required editing NifModel and I thought this was ugly and trashed it. I thought my current solution using qApp->allWidgets() was functionally equivalent, but in actuality this is enabling/disabling all widgets named 'tLOD' across all windows based on the last mesh loaded.

hexabits referenced this issue in hexabits/nifskope May 18, 2014
Issue #22

Went back to previous method of disabling slider via signal because I
could not detect the proper tLOD bar for the window that the mesh
belongs to.  The qApp method was enabling/disabling all sliders in all
windows.
@hexabits hexabits mentioned this issue Jun 12, 2014
@hexabits hexabits modified the milestones: v1.2.0, v2.0.0 Nov 2, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant