Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

droumis
Copy link
Member

@droumis droumis commented Jul 24, 2024

Closes #6340

GMT20240724-192251_Clip_Demetris.Roumis.s.Clip.07_24_2024.mp4

@droumis
Copy link
Member Author

droumis commented Jul 24, 2024

@holoviz-developers, I'm seeking feedback on default styling of box handles and inverted highlighting

e.g.:

            tool.overlay.handles.all.hover_fill_color = "grey"
            tool.overlay.handles.all.hover_fill_alpha = 0.25
            tool.overlay.handles.all.hover_line_alpha = 0
            tool.overlay.handles.all.fill_alpha = 0.1
            tool.overlay.handles.all.line_alpha = 0.25

@droumis droumis added tag: backend: bokeh type: enhancement Minor feature or improvement to an existing feature labels Jul 24, 2024
Copy link

codecov bot commented Jul 31, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.49%. Comparing base (a28189d) to head (d016b5d).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6341      +/-   ##
==========================================
+ Coverage   88.48%   88.49%   +0.01%     
==========================================
  Files         323      323              
  Lines       68162    68179      +17     
==========================================
+ Hits        60313    60337      +24     
+ Misses       7849     7842       -7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ahuang11
Copy link
Collaborator

Styling looks good to me!

@droumis droumis marked this pull request as ready for review August 1, 2024 14:33
@droumis
Copy link
Member Author

droumis commented Aug 5, 2024

@hoxbro , is this good to merge?

Copy link
Member

@hoxbro hoxbro left a comment

Choose a reason for hiding this comment

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

Does this change the default behavior?

If it does, I think we need to warn people if they have a version lower than Bokeh 3.5 and actively set these new parameters.

Comment on lines 118 to 125
use_handles = param.Boolean(default=True, doc="""
Whether to display handles""")

start_gesture = param.ObjectSelector(default='tap', objects=['pan', 'tap', 'none'],
doc="""Gesture to start a range selection""")

inverted = param.Boolean(default=True, doc="""
Whether to invert the highlighting of the range selection""")
Copy link
Member

Choose a reason for hiding this comment

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

Maybe add information that this is only available from Bokeh 3.5

Copy link
Member Author

Choose a reason for hiding this comment

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

I added a note in each of the docstrings

@droumis
Copy link
Member Author

droumis commented Aug 7, 2024

If it does, I think we need to warn people if they have a version lower than Bokeh 3.5 and actively set these new parameters.

I added a warning if bokeh34. Is this what you are suggesting?

        if bokeh34:
            warn("RangeTool styling and interactivity options updated in Bokeh 3.5.")

@hoxbro hoxbro enabled auto-merge (squash) August 8, 2024 14:00
@hoxbro hoxbro merged commit 59d81fa into main Aug 8, 2024
13 checks passed
@hoxbro hoxbro deleted the rangetool-boxselect branch August 8, 2024 17:10
@droumis droumis added this to the 1.19.2 milestone Sep 27, 2024
@hoxbro hoxbro modified the milestones: 1.19.2, 1.20.0 Nov 1, 2024
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 11, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
tag: backend: bokeh type: enhancement Minor feature or improvement to an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add recent Bokeh 3.5 enhancements to RangeToolLink
3 participants