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

[BUG] Grid covering axes #4202

Closed
putianyi889 opened this issue May 15, 2022 · 0 comments · Fixed by #4960
Closed

[BUG] Grid covering axes #4202

putianyi889 opened this issue May 15, 2022 · 0 comments · Fixed by #4960
Labels
bug GR PGFPlotsX

Comments

@putianyi889
Copy link
Contributor

Details

plot(1:5,1:5,1:5, xlims=(1,5), ylims=(1,5), zlims=(1,5), framestyle=:box,foreground_color_grid=:red, gridalpha=1, thickness_scaling=5, color=:transparent)

on pgfplotsx():

download

Backends

This bug occurs on ( insert x below )

Backend yes no untested
gr (default) x
pyplot x
plotlyjs x
pgfplotsx x
unicodeplots x
inspectdr x
gaston x

Versions

Plots.jl version: 1.29.0
Backend version (]st -m <backend(s)>): GR 0.64.2, PGFPlotsX 1.5.0
Output of versioninfo():

Julia Version 1.7.2
Commit bf53498635 (2022-02-06 15:21 UTC)
Platform Info:
  OS: Windows (x86_64-w64-mingw32)
  CPU: AMD Ryzen 7 4800H with Radeon Graphics
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-12.0.1 (ORCJIT, znver2)
Environment:
  JULIA_EDITOR = code
  JULIA_NUM_THREADS = 16
@dd0 dd0 mentioned this issue Jun 18, 2024
Open
dd0 added a commit to dd0/Plots.jl that referenced this issue Jun 18, 2024
@dd0
GR: Draw all gridlines before axis
02f1f02 
Separates grid drawing from the remainder of gr_draw_axis, so that all
gridlines are drawn below the axis. Previously, y-axis gridlines were
drawn after the x-axis and could overlap it.

Fixes issue JuliaPlots#4202 for GR.
dd0 added a commit to dd0/Plots.jl that referenced this issue Jun 18, 2024
@dd0
GR: Draw all gridlines before axis
f64c0a0 
Separates grid drawing from the remainder of gr_draw_axis, so that all
gridlines are drawn below the axis. Previously, y-axis gridlines were
drawn after the x-axis and could overlap it.

Fixes issue JuliaPlots#4202 for GR.
dd0 added a commit to dd0/Plots.jl that referenced this issue Jun 18, 2024
@dd0
GR: Draw all gridlines before axis
849dc95 
Separates grid drawing from the remainder of gr_draw_axis, so that all
gridlines are drawn below the axis. Previously, y-axis gridlines were
drawn after the x-axis and could overlap it.

Fixes issue JuliaPlots#4202 for GR.
@dd0 dd0 mentioned this issue Jun 18, 2024
Merged
7 tasks
dd0 added a commit to dd0/Plots.jl that referenced this issue Jul 12, 2024
@dd0
GR: Draw all gridlines before axis
3c6991b 
Separates grid drawing from the remainder of gr_draw_axis, so that all
gridlines are drawn below the axis. Previously, y-axis gridlines were
drawn after the x-axis and could overlap it.

Fixes issue JuliaPlots#4202 for GR.
@dd0 dd0 mentioned this issue Jul 12, 2024
Merged
7 tasks
BeastyBlacksmith pushed a commit that referenced this issue Jul 13, 2024
@dd0
GR: Draw all gridlines before axis (#4945)
8b9bcf2 
Separates grid drawing from the remainder of gr_draw_axis, so that all
gridlines are drawn below the axis. Previously, y-axis gridlines were
drawn after the x-axis and could overlap it.

Fixes issue #4202 for GR.
BeastyBlacksmith pushed a commit that referenced this issue Jul 13, 2024
@dd0
GR: Draw all gridlines before axis (v2) (#4960)
617b8aa 
* GR: Draw all gridlines before axis

Separates grid drawing from the remainder of gr_draw_axis, so that all
gridlines are drawn below the axis. Previously, y-axis gridlines were
drawn after the x-axis and could overlap it.

Fixes issue #4202 for GR.

* Added qualified PlotsBase names missing from v1 port
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug GR PGFPlotsX
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

GR: Draw all gridlines before axis (v2) dd0/Plots.jl
3 participants
@t-bltg @BeastyBlacksmith @putianyi889