-
Notifications
You must be signed in to change notification settings - Fork 18
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
Fix Heretic Profiles and Search #166
Fix Heretic Profiles and Search #166
Conversation
Additionally here are some merge check failures. I'm honestly not certain if those come from this PR or the canonical branch; didn't check so am not assigning blame there. |
Whurks now
|
…y moved them to LLProfileImagePicker.
Seems like all that was already working. I didn't touch anywhere near those menus. |
Create Pick has been under the Places panel since Viewer2's inception 15 years ago. Editing display name is achieved by double clicking the profile name as it has been since these profiles were developed for Alchemy 10 years ago. |
Adapting to this hackish |
…ht click image to activate menu.
Right clicking the profile images now creates a flyout menu with Upload, Change, and Remove image options. |
LGTM |
There are two accordian tabs there, one for Picks one for Classifieds. If there are no items, the accordian tab for that category is hidden. This is a carryover from the original V2 panel design. |
All's good then, thanks for the requested changes! Been a long time since I v2'd and it shows lol |
Incidentally, if there are no groups (and it isn't your profile) the Groups accordion tab is also hidden. Picks could be rewritten to be controllable from the profile (a la Firestorm) but that requires duplicating a fair amount of work that still resides in Places (and LLProfilePicks). There would be triplicate functionality across the project at that point and excessive technical debt maintaining it. Places itself is the LLSideTrayPanelContainer bitch from hell that just won't die. Tried decoupling it years ago, and it's just a pain the ass. Likely why it survived the original side panel container purge. At this point though, all functionality is restored to Heretic profiles. |
Reinstate objectively better Alchemy profiles and search. Ain't no thang.