Skip to content
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

feat!: reworked opacity options #680

Merged
merged 1 commit into from
Oct 8, 2023
Merged

feat!: reworked opacity options #680

merged 1 commit into from
Oct 8, 2023

Conversation

tomasklaen
Copy link
Owner

These options have been removed:

timeline_opacity
timeline_chapters_opacity
volume_opacity
speed_opacity
menu_opacity
menu_parent_opacity
top_bar_title_opacity
window_border_opacity
curtain_opacity

and replaced with one comma delimited opacity list to override all of the above and some new default opacity values.

Example:

opacity=timeline=0.5,title=0.5

List of opacity values available for override and their current defaults:

timeline=.9
position=1     # timeline progress or line
chapters=0.8
slider=0.9     # background of all sliders, such as volume
slider_gauge=1 # value bar of all sliders
speed=0.6
menu=1
submenu=0.4
border=1       # window border
title=1        # window title
tooltip=1      # tooltip background
thumbnail=1    # thumbnail border
curtain=0.5

closes #584, closes #636, closes #675

These options have been removed:

```
timeline_opacity
timeline_chapters_opacity
volume_opacity
speed_opacity
menu_opacity
menu_parent_opacity
top_bar_title_opacity
window_border_opacity
curtain_opacity
```

and replaced with one comma delimited `opacity` list to override all of the above and some new default opacity values.

Example:

```
opacity=timeline=0.5,title=0.5
```

List of opacity values available for override and their current defaults:

```
timeline=.9
position=1     # timeline progress or line
chapters=0.8
slider=0.9     # background of all sliders, such as volume
slider_gauge=1 # value bar of all sliders
speed=0.6
menu=1
submenu=0.4
border=1       # window border
title=1        # window title
tooltip=1      # tooltip background
thumbnail=1    # thumbnail border
curtain=0.5
```

closes #584, closes #636, closes #675
@tomasklaen
Copy link
Owner Author

I wonder if speed should just use slider value.

@christoph-heinrich
Copy link
Contributor

The speed control is aesthetically (and functionally) very different from the volume slider, so it makes sense to be separate.

@tomasklaen tomasklaen merged commit b800aeb into main Oct 8, 2023
@tomasklaen tomasklaen deleted the opacity_rework branch October 9, 2023 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add option to hide background for tooltips Changing opacity of timeline progress
2 participants