Skip to content
This repository was archived by the owner on Jan 14, 2025. It is now read-only.

Conversation

@wyrnat
Copy link

@wyrnat wyrnat commented Dec 11, 2023

untested as the current master version is not working for me

what

  • display human readable names for the Environment combobox

why

  • user is able to set the displayed name without restriction from filesystem naming
  • what was the use of the object variable "name" in Environment anyway? Please correct me if I am wrong, but I could not find the purpose

how

  • load all environments from the env_list and gather "name" from each. gather all names into a list
  • set modified list to GUI object for display
  • catch a missing object variable or a faulty environment. object variable should not be mandatory and faulty environment will be handled when user loads it for usage

closes #14

wenatuhs added a commit that referenced this pull request Dec 12, 2023
@wenatuhs
Copy link
Contributor

@wyrnat Thanks so much for the contribution! It has been years since this PR (sorry for the inactivity in this repo! Recent activities have been happening in the new location) so I feel maybe we have to do some work on the code to adapt it to the new code base. I'll leave the PR as is and archive this old repo (to not cause further confusion). Talk to you in the new repo soon!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

appearance: display Environment class variable "name" in the Environment Combobox

2 participants