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.

Sign up for GitHub

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

Jump to bottom

Make line width equal to point's stroke width #798

Merged
merged 9 commits into from
Jun 28, 2023
Merged

Make line width equal to point's stroke width #798

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
9a973fc
Make line width equal to point's stroke width. Add their defaults for…
OLarionova-HORIS Jun 22, 2023
dff3070
Make `fill`=<background color> for all point geoms.
OLarionova-HORIS Jun 26, 2023
b95864b
Cleanup: remove DefaultGeomTheme.BASE.
OLarionova-HORIS Jun 26, 2023
2bd5c8b
SmoothGeom: draw line over the confidence interval. Simplify LinesHel…
OLarionova-HORIS Jun 26, 2023
ab97c0e
Restore commented code.
OLarionova-HORIS Jun 26, 2023
911118e
Add fixed colors structure.
OLarionova-HORIS Jun 26, 2023
2079efc
Set the default `fill` for crossbar and box_plot based on background …
OLarionova-HORIS Jun 26, 2023
71c2fa9
Enlarge default values for size, linewidth, stroke.
OLarionova-HORIS Jun 26, 2023
39dc2d3
Return alpha=0.1 for some geometries.
OLarionova-HORIS Jun 27, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add fixed colors structure.
  • Loading branch information
@OLarionova-HORIS
OLarionova-HORIS committed Jun 26, 2023
commit 911118e6263df4ddabd2ebc00ff0f2d3110c22f7
16 changes: 13 additions & 3 deletions ...le/src/commonMain/kotlin/jetbrains/datalore/plot/builder/defaultTheme/DefaultGeomTheme.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,10 +53,21 @@ internal class DefaultGeomTheme private constructor(
}
}

private class FixedColors(geomKind: GeomKind) {
val color = if (geomKind == GeomKind.SMOOTH) {
Color.MAGENTA
} else {
Color.PACIFIC_BLUE
}
val fill = Color.PACIFIC_BLUE
}

// defaults for geomKind
fun forGeomKind(geomKind: GeomKind, inheritedColors: InheritedColors): GeomTheme {
var color = Color.PACIFIC_BLUE
var fill = Color.PACIFIC_BLUE
val fixedColors = FixedColors(geomKind)

var color = fixedColors.color
var fill = fixedColors.fill
var alpha = 1.0
var size = 0.5
var lineWidth = 0.5
Expand Down Expand Up @@ -123,7 +134,6 @@ internal class DefaultGeomTheme private constructor(
}

GeomKind.SMOOTH -> {
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is bad: alpha = 1.5

                GeomKind.SMOOTH -> {
                    color = Color.MAGENTA

We don't want to use constant values no longer. Should be a field in BASE (but still get rid of the BASE).

color = Color.MAGENTA
fill = inheritedColors.lineColor()
alpha = 0.15
}
Expand Down