Skip to content
Snippets Groups Projects
Commit 7b24b7a7 authored by Charan DSouza's avatar Charan DSouza
Browse files

Fixed the bug #65.

Analysis:
This bug is there for following reasons.
1.ElectrodeButton for channels is directly added to the ChannelMappingEditor component.
2.When we drag the button we set the visibility of the lastHoverButton which makes it visible and shows on top of other components.
3. Also finding the correct button Logic is not proper as we are not considering the buttons which are scrolled up and invisible.

Solution:
1. Create Viewport for holding the ElectrodeButton for channels.
2. Also create a Holder Component to hold all ElectrodeButton.
3. Add the ElectrodeButton to Holder Component and set the size of the Holder Component to fit all buttons.
4. Set setViewPosition for Viewport based on the scroll and drag.

More Improvement:
1. We can remove upButton and downButton.
2. Viewport has its own scrolling which can be used for scrolling. Currently its been hidden using code "electrodeButtonViewport->setScrollBarsShown(false,false,true,true);"
parents 19706f46 f754c628
Branches
Tags
No related merge requests found
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment