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

Show file tree
Hide file tree
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
Next Next commit
Improve linetype experience (LPK-220).
  • Loading branch information
@ASmirnov-HORIS
ASmirnov-HORIS committed May 8, 2024
commit 7de207e18f5030430b012c013acc0966fb97b5d7
1 change: 1 addition & 0 deletions future_changes.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,3 +8,4 @@

- Livemap: improve "tiles" documentation [[#1093](https://github.com/JetBrains/lets-plot/issues/1093)].
- Undesired vertical scroller when displaying `gggrid` in Jupyter notebook.
- Improve border line type experience [[LPK-220](https://github.com/JetBrains/lets-plot-kotlin/issues/220)].
10 changes: 5 additions & 5 deletions .../commonMain/kotlin/org/jetbrains/letsPlot/core/plot/base/render/linetype/NamedLineType.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,14 +16,14 @@ enum class NamedLineType(val code: Int, private val myDashArray: List<Double>?)
override val isSolid: Boolean
get() = true
},
DASHED(2, listOf(4.3, 4.3)),
DOTTED(3, listOf(1.0, 3.2)),
DASHED(2, listOf(4.0, 4.0)),
DOTTED(3, listOf(1.0, 3.0)),
@Suppress("SpellCheckingInspection")
DOTDASH(4, listOf(1.0, 3.2, 4.3, 3.2)),
DOTDASH(4, listOf(1.0, 3.0, 4.0, 3.0)),
@Suppress("SpellCheckingInspection")
LONGDASH(5, listOf(7.4, 3.2)),
LONGDASH(5, listOf(7.0, 3.0)),
@Suppress("SpellCheckingInspection")
TWODASH(6, listOf(2.4, 2.4, 6.4, 2.4));
TWODASH(6, listOf(2.0, 2.0, 6.0, 2.0));

override val isSolid: Boolean
get() = false
Expand Down
4 changes: 2 additions & 2 deletions ...vemap/src/commonTest/kotlin/org/jetbrains/letsPlot/core/plot/livemap/PathConverterTest.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ class PathConverterTest {

matcher!!
.strokeWidth(eq(1.1))
.lineDash(vectorEq(listOf(4.73, 4.73)))
.lineDash(vectorEq(listOf(4.4, 4.4)))

assertMapObject()
}
Expand All @@ -61,7 +61,7 @@ class PathConverterTest {

matcher!!
.strokeWidth(eq(4.4))
.lineDash(vectorEq(listOf(18.92, 18.92)))
.lineDash(vectorEq(listOf(17.6, 17.6)))

assertMapObject()
}
Expand Down