-
Notifications
You must be signed in to change notification settings - Fork 679
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
Fixes #2489. Create new ScrollBar based on a new Scroll and remove ScrollBarView/ScrollView #3498
base: v2_develop
Are you sure you want to change the base?
Conversation
I would appreciate the |
Ooooh! I'm super excited to review this! Will do asap. |
@tig how about you creating a new |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really slick!!
I've got a PR with some changes that you might like. I'll submit it vs. making a lot of comments.
Tweaks and suggestions
You mean being able to set the Normal.Background/Forground color for the View itself? Like the colorpicker in |
Yes or for the view passed on the |
@tig did you reorganize by placing the backfields before the properties by hand, or through some automatic system? I normally use R#'s Ctrl+E+C and it's not reformatting that way. |
By hand. It's still broken. |
Greet idea. I'll do it as part of #3376 where I've already completely re-done I also added the abilty to just click on any view in the Scenario and |
I just submitted another PR with quesitons / comments inline. |
Suggestions and questions
IMO, View sub-classes should not have the word "View" in them as a general principle. `
|
I added highlight effect in the slider. I didn't used the WindowsTerminal_rJyd5R3RZY.mp4PS: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Took a look at it. Doesn't quite do what you were hoping, but fix is simple. :)
…erformance in the Position property.
Not needed I just override the |
I'm not explaining myself well. If you move the mouse really fast outside of the view that hosts the scroll bar, mouse tracking within the scroll bar stops. |
Ah maybe because the |
Good catch. Man, it would be awesome if we had some automated UI testing to add this sort of thing to. Sucks only having 24 hours in a day, huh? 😅 |
Width = Dim.Auto (DimAutoStyle.Content, 1); | ||
Height = Dim.Auto (DimAutoStyle.Content, 1); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tig if a derived view want to constraint the Width
and the Height
to avoid the user set inappropriate values, how that can be done, if they aren't virtual?
Fixes
ScrollBar
based on a newScroll
and removeScrollBarView/ScrollView
#2489Proposed Changes/Todos
Pull Request checklist:
CTRL-K-D
to automatically reformat your files before committing.dotnet test
before commit///
style comments)