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

Add 'linetype' for theme elements #1072

Merged
merged 14 commits into from
Apr 3, 2024
Merged

Add 'linetype' for theme elements #1072

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
57bdaf1
Add interfaces to set stroke-dasharray attribute for svg elements.
OLarionova-HORIS Apr 2, 2024
55bc212
Add 'linetype' parsing as theme option.
OLarionova-HORIS Apr 2, 2024
cf3624e
For `plot_background`.
OLarionova-HORIS Apr 2, 2024
ac65c21
For `panel_background, panel_border, panel_grid_major/minor`.
OLarionova-HORIS Apr 2, 2024
79e10db
For `legend_background, strip_background`.
OLarionova-HORIS Apr 2, 2024
3898e7f
For `axis_line, axis_ticks`.
OLarionova-HORIS Apr 2, 2024
d791150
For `axis_line, axis_ticks`.
OLarionova-HORIS Apr 2, 2024
510397b
Add parameter 'linetype' to py functions.
OLarionova-HORIS Apr 2, 2024
50368f5
Renaming.
OLarionova-HORIS Apr 2, 2024
a63c36c
Cleanup code.
OLarionova-HORIS Apr 2, 2024
2da45ca
Fix docstrings.
OLarionova-HORIS Apr 2, 2024
da47377
Cleanup linetype parse function.
OLarionova-HORIS Apr 2, 2024
9339476
Add notebook.
OLarionova-HORIS Apr 3, 2024
2dbd3d1
Rename notebook. Update future_changes.md.
OLarionova-HORIS Apr 3, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
Prev Previous commit
Next Next commit
For plot_background.
  • Loading branch information
@OLarionova-HORIS
OLarionova-HORIS committed Apr 2, 2024
commit cf3624e9c4357b6fda30b69a2eda252ad7fee993
2 changes: 2 additions & 0 deletions plot-base/src/commonMain/kotlin/org/jetbrains/letsPlot/core/plot/base/theme/PlotTheme.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,13 +8,15 @@ package org.jetbrains.letsPlot.core.plot.base.theme
import org.jetbrains.letsPlot.commons.values.Color
import org.jetbrains.letsPlot.core.plot.base.layout.TextJustification
import org.jetbrains.letsPlot.core.plot.base.layout.Thickness
import org.jetbrains.letsPlot.core.plot.base.render.linetype.LineType

interface PlotTheme {
fun showBackground(): Boolean
fun showMessage(): Boolean
fun backgroundColor(): Color
fun backgroundFill(): Color
fun backgroundStrokeWidth(): Double
fun backgroundRectLineType(): LineType
fun titleStyle(): ThemeTextStyle
fun subtitleStyle(): ThemeTextStyle
fun captionStyle(): ThemeTextStyle
Expand Down
5 changes: 4 additions & 1 deletion ...uilder/src/commonMain/kotlin/org/jetbrains/letsPlot/core/plot/builder/PlotSvgComponent.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ import org.jetbrains.letsPlot.core.plot.base.layout.TextJustification.Companion.
import org.jetbrains.letsPlot.core.plot.base.layout.TextJustification.Companion.applyJustification
import org.jetbrains.letsPlot.core.plot.base.layout.Thickness
import org.jetbrains.letsPlot.core.plot.base.render.svg.MultilineLabel
import org.jetbrains.letsPlot.core.plot.base.render.svg.StrokeDashArraySupport
import org.jetbrains.letsPlot.core.plot.base.render.svg.SvgComponent
import org.jetbrains.letsPlot.core.plot.base.render.svg.Text.HorizontalAnchor
import org.jetbrains.letsPlot.core.plot.base.render.svg.Text.VerticalAnchor
Expand Down Expand Up @@ -251,7 +252,9 @@ class PlotSvgComponent constructor(
backgroundBorder.apply {
fillColor().set(Color.TRANSPARENT)
strokeColor().set(plotTheme.backgroundColor())
strokeWidth().set(plotTheme.backgroundStrokeWidth())
val width = plotTheme.backgroundStrokeWidth()
strokeWidth().set(width)
StrokeDashArraySupport.apply(this, width, plotTheme.backgroundRectLineType())
d().set(SvgPathDataBuilder().rect(backgroundRect).build())
}
}
Expand Down
5 changes: 5 additions & 0 deletions ...mmonMain/kotlin/org/jetbrains/letsPlot/core/plot/builder/defaultTheme/DefaultPlotTheme.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ package org.jetbrains.letsPlot.core.plot.builder.defaultTheme

import org.jetbrains.letsPlot.commons.values.Color
import org.jetbrains.letsPlot.core.plot.base.layout.TextJustification
import org.jetbrains.letsPlot.core.plot.base.render.linetype.LineType
import org.jetbrains.letsPlot.core.plot.base.theme.PlotTheme
import org.jetbrains.letsPlot.core.plot.base.theme.ThemeTextStyle
import org.jetbrains.letsPlot.core.plot.builder.defaultTheme.values.ThemeOption.Elem
Expand Down Expand Up @@ -49,6 +50,10 @@ internal class DefaultPlotTheme(
return getNumber(getElemValue(backgroundKey), Elem.SIZE)
}

override fun backgroundRectLineType(): LineType {
return getLineType(getElemValue(backgroundKey))
}

override fun titleStyle(): ThemeTextStyle {
return getTextStyle(getElemValue(titleKey))
}
Expand Down
2 changes: 2 additions & 0 deletions ...c/commonMain/kotlin/org/jetbrains/letsPlot/core/plot/builder/defaultTheme/DefaultTheme.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ import org.jetbrains.letsPlot.core.plot.builder.defaultTheme.values.ThemeOption
import org.jetbrains.letsPlot.core.plot.builder.defaultTheme.values.ThemeOption.Elem.BLANK
import org.jetbrains.letsPlot.core.plot.builder.defaultTheme.values.ThemeOption.Elem.COLOR
import org.jetbrains.letsPlot.core.plot.builder.defaultTheme.values.ThemeOption.Elem.FILL
import org.jetbrains.letsPlot.core.plot.builder.defaultTheme.values.ThemeOption.Elem.LINETYPE
import org.jetbrains.letsPlot.core.plot.builder.defaultTheme.values.ThemeOption.Elem.Margin
import org.jetbrains.letsPlot.core.plot.builder.defaultTheme.values.ThemeOption.Elem.SIZE
import org.jetbrains.letsPlot.core.plot.builder.presentation.DefaultFontFamilyRegistry
Expand Down Expand Up @@ -67,6 +68,7 @@ class DefaultTheme internal constructor(
FILL to containerTheme.plot.backgroundFill(),
COLOR to this.plot.backgroundColor(),
SIZE to this.plot.backgroundStrokeWidth(),
LINETYPE to this.plot.backgroundRectLineType()
)
}

Expand Down
2 changes: 1 addition & 1 deletion ...n/kotlin/org/jetbrains/letsPlot/core/plot/builder/subPlots/CompositeFigureSvgComponent.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ class CompositeFigureSvgComponent constructor(
strokeColor().set(plotTheme.backgroundColor())
val width = plotTheme.backgroundStrokeWidth()
strokeWidth().set(width)
StrokeDashArraySupport.apply(this, width, plotTheme.backgroundStrokeLineType())
StrokeDashArraySupport.apply(this, width, plotTheme.backgroundRectLineType())
})
}
}
Expand Down