Skip to content

Conversation

@PJacek
Copy link
Contributor

@PJacek PJacek commented Jun 9, 2022

Description of the problem being solved:

Fix multiple tooltips displayed over each other, specifically when hovering over a dropdown control.

ControlHost remembers the currently active control in selControl and can inform it's controls via additional parameter noTooltip passed to Draw methods whether they should draw tooltips. Additional parameter in DrawControls allows to pass the active control to lower levels in cases of nested controls (e.g. CalcSectionControl)

Move bandit and pantheon dropdowns to the configuration tab.

This frees some space in the side panel. Loading old build should work correctly and the attributes are saved in the Build node as well for backwards compatibility.

Before screenshot:

image

After screenshot:

image

@PJacek PJacek added bug Something isn't working user-interface Changes that only affect the UI labels Jun 9, 2022
Copy link
Member

@Wires77 Wires77 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This probably should've been moved into config options a long time ago

@Wires77 Wires77 changed the title UI improvements Fix overlapping tooltips, move bandit and pantheon options into the Config tab Jul 9, 2022
@Wires77 Wires77 merged commit 5750d0f into PathOfBuildingCommunity:dev Jul 9, 2022
@Wires77 Wires77 removed the bug Something isn't working label Jul 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

user-interface Changes that only affect the UI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants