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

Improve border line type experience #1106

Merged
merged 8 commits into from
May 24, 2024
Merged

Improve border line type experience #1106

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
7de207e
Improve linetype experience (LPK-220).
ASmirnov-HORIS May 8, 2024
544bbc5
Merge branch 'refs/heads/master' into LPK-220
ASmirnov-HORIS May 8, 2024
34728d9
Fix conflicts after merging with master.
ASmirnov-HORIS May 20, 2024
9931a32
LPK-220: Change linewidth calculation for the midline of the boxplot.
ASmirnov-HORIS May 20, 2024
fcc8865
LPK-220: Refactor code and fix linewidth calculation for the crossbars.
ASmirnov-HORIS May 20, 2024
788b528
LPK-220: Refactor code.
ASmirnov-HORIS May 24, 2024
85e0660
Merge branch 'master' into LPK-220
ASmirnov-HORIS May 24, 2024
9d71292
LPK-220: Another small refactor in BoxHelper.
ASmirnov-HORIS May 24, 2024
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
LPK-220: Change linewidth calculation for the midline of the boxplot.
  • Loading branch information
@ASmirnov-HORIS
ASmirnov-HORIS committed May 20, 2024
commit 9931a32319d28c9398732b19bc62fe58d8ec4857
4 changes: 1 addition & 3 deletions plot-base/src/commonMain/kotlin/org/jetbrains/letsPlot/core/plot/base/geom/util/BoxHelper.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -51,13 +51,11 @@ object BoxHelper {
DoubleVector(x - width / 2, middle),
DoubleVector(x + width / 2, middle),
p
) ?: continue
) { AesScaling.strokeWidth(it) * fatten } ?: continue

// TODO: use strokeScale in createLine() function
// adjust thickness
require(line is SvgShape)
val thickness = line.strokeWidth().get()!!
line.strokeWidth().set(thickness * fatten)

root.add(line)
}
Expand Down