DOC: Slightly further improve arrowstyle demo#29801
Conversation
|
Note: doc build failure is likely unrelated, see #29802. |
story645
left a comment
story645
left a comment
There was a problem hiding this comment.
I kinda want a box around each entry but not sure that it would make things clearer.
| l, = ax.plot(1, 0, "o", color="grey") | ||
| ax.annotate(stylename, (1, 0), (0, 0), | ||
| size="large", color="tab:blue", ha="center", va="center", | ||
| l, = ax.plot(1.25, 0, "o", color="lightgrey") |
There was a problem hiding this comment.
can this be like one or two shades darker?
timhoffm
force-pushed
the
doc-arrowstyle
branch
from
9c01223 to
3323b92
Compare
Not sure that's better. It looks more imbalanced because the number of parameters vary strongly.
|
|
what about a faded or dotted grid? I like the boundaries but I get how here it starts looking cluttered |
|
Feel free to play around if you like. But I think if we add boxes, we need another approach to the titles. They don't really make sense in large boxes. |
|
The boxes don't really work there. At most, I'd say a line down the middle could work. |
|
I wouldn't include boxes. I'd add a bit more wspace between the columns. Why are AngleA and AngleB all specified in these but set to zero? The constructor for the axs is needlessly complex. |
galleries/examples/text_labels_and_annotations/fancyarrow_demo.py
Outdated
Show resolved
Hide resolved
Do you mean the parameter blocks in the rendered figure? That's
|
- use straight lines - use monospace font for the style names - adjust positions and colors so that the arrows stand out more
timhoffm
force-pushed
the
doc-arrowstyle
branch
from
3323b92 to
1c39a10
Compare
jklymak
left a comment
jklymak
left a comment
There was a problem hiding this comment.
Feel free to self merge if you feel you've dealt with everything.
…801-on-v3.10.x Backport PR #29801 on branch v3.10.x (DOC: Slightly further improve arrowstyle demo)
…801-on-v3.10.1-doc Backport PR #29801 on branch v3.10.1-doc (DOC: Slightly further improve arrowstyle demo)
4 participants
Uh oh!
There was an error while loading. Please reload this page.