Opened 10 years ago
Closed 18 months ago
#14020 closed defect (invalid)
interact with layout is broken
Reported by: | Andrey Novoseltsev | Owned by: | jason, mpatel, was |
---|---|---|---|
Priority: | major | Milestone: | sage-duplicate/invalid/wontfix |
Component: | notebook | Keywords: | interact sd48 |
Cc: | Jason Grout, Karl-Dieter Crisman, Frédéric Chapoton | Merged in: | |
Authors: | Reviewers: | Dima Pasechnik | |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by )
Consider this example:
@interact(layout={ "top": [ ["u_lines"], ["v_lines"], ] }) def _( u_lines=slider([1..50], default=10, label=r"$u$-lines:", display_value=False), v_lines=slider([1..50], default=10, label=r"$v$-lines:", display_value=False), ): print (u_lines, v_lines)
With 5.7.beta1 (and I believe the problem existed for a while):
- The vertical spacing between elements is HUGE - if I remove layout (which is supposed to reproduce default placement here anyway), it is much more reasonable. This makes it next to impossible to create interacts to show in class that will have all controls and graphs on one screen in a reasonable size.
- Math is not processed in labels. Again, everything is OK without layout.
- Values of sliders are not actually updated - this is due to
display_value=False
with or without layout.
In Cell Server there are no issues with layout, but display_value=False
does not work - the value is still shown.
Attachments (1)
Change History (15)
comment:1 Changed 10 years ago by
Cc: | Karl-Dieter Crisman added |
---|
comment:2 Changed 9 years ago by
comment:3 Changed 9 years ago by
Keywords: | sd48 added |
---|
comment:4 Changed 9 years ago by
Milestone: | sage-5.11 → sage-5.12 |
---|
comment:5 Changed 9 years ago by
Milestone: | sage-6.1 → sage-6.2 |
---|
comment:6 Changed 9 years ago by
Milestone: | sage-6.2 → sage-6.3 |
---|
comment:7 Changed 8 years ago by
Milestone: | sage-6.3 → sage-6.4 |
---|
comment:8 Changed 3 years ago by
Milestone: | sage-6.4 → sage-duplicate/invalid/wontfix |
---|---|
Status: | new → needs_review |
sagenb is now deprecated
comment:9 Changed 3 years ago by
Does this work on Jupyter? We could repurpose the ticket. And the syntax is "supposed to work" on platforms ...
comment:10 Changed 2 years ago by
Cc: | Frédéric Chapoton added |
---|
Proposing to close all sagenb tickets as outdated, so that all remaining open tickets in the notebook component are about the Jupyter notebook.
comment:11 Changed 18 months ago by
Reviewers: | → Dima Pasechnik |
---|---|
Status: | needs_review → positive_review |
comment:12 follow-up: 13 Changed 18 months ago by
Status: | positive_review → needs_info |
---|
Again, does this work with Jupyter? If not, we might as well just turn this ticket into a Jupyter one to conform with comment:10. Else close, yes.
comment:13 Changed 18 months ago by
Description: | modified (diff) |
---|---|
Status: | needs_info → positive_review |
Replying to kcrisman:
Again, does this work with Jupyter? If not, we might as well just turn this ticket into a Jupyter one to conform with comment:10. Else close, yes.
yes, it does work with Jupyter, see attachment
comment:14 Changed 18 months ago by
Resolution: | → invalid |
---|---|
Status: | positive_review → closed |
https://github.com/sagemath/sagenb/pull/168 solves the first two bullets