Rename the switch--switch component class to switch--slider

Following on from #1751: originally Switch was called Checkbox and the
moving part was, for the component class, called a checkbox--switch; after
renaming the widget to Switch that component class ended up being
switch--switch; which wasn't ideal.

We decided to go with it as-is, but I just realised that internally the code
calls it a slider. So this leans into that and I'm renaming the component
class switch--slider. This removes the doubling-up of the name and also
makes the code more consistent.
This commit is contained in:
Dave Pearson
2023-02-09 15:36:04 +00:00
parent 989e71e749
commit 1956f7f9d7
2 changed files with 6 additions and 6 deletions

View File

@@ -22,7 +22,7 @@ Switch {
background: darkslategrey;
}
#custom-design > .switch--switch {
#custom-design > .switch--slider {
color: dodgerblue;
background: darkslateblue;
}

View File

@@ -31,12 +31,12 @@ class Switch(Widget, can_focus=True):
"""
COMPONENT_CLASSES: ClassVar[set[str]] = {
"switch--switch",
"switch--slider",
}
"""
| Class | Description |
| :- | :- |
| `switch--switch` | Targets the switch of the switch. |
| `switch--slider` | Targets the slider of the switch. |
"""
DEFAULT_CSS = """
@@ -48,7 +48,7 @@ class Switch(Widget, can_focus=True):
padding: 0 2;
}
Switch > .switch--switch {
Switch > .switch--slider {
background: $panel-darken-2;
color: $panel-lighten-2;
}
@@ -65,7 +65,7 @@ class Switch(Widget, can_focus=True):
}
Switch.-on > .switch--switch {
Switch.-on > .switch--slider {
color: $success;
}
"""
@@ -128,7 +128,7 @@ class Switch(Widget, can_focus=True):
self.set_class(slider_pos == 1, "-on")
def render(self) -> RenderableType:
style = self.get_component_rich_style("switch--switch")
style = self.get_component_rich_style("switch--slider")
return ScrollBarRender(
virtual_size=100,
window_size=50,