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.

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

interact bug -- drop down menu default doesn't show default value #2530

Closed
williamstein opened this issue Mar 15, 2008 · 4 comments
Closed

Comments

@williamstein
Copy link
Contributor

@interact
def _(f = (33,[1..100])):
    pass

The drop down should be set to 33 but isn't.

Component: notebook

Issue created by migration from https://trac.sagemath.org/ticket/2530

@williamstein
Copy link
Contributor Author

comment:1

Attachment: sage-2530.patch.gz

The attached patch fixes some minor issues with selectors with interact:

  1. the default for a drop-down menu is now correct,

  2. buttons with strings are no longer have quotes,

  3. the default value for selectors is the actual value rather than the index into the list of values, which is much much more natural.

@TimothyClemans
Copy link
Mannequin

TimothyClemans mannequin commented Mar 16, 2008

comment:2

I tested this, and it looks good to me.

@sagetrac-mabshoff sagetrac-mabshoff mannequin modified the milestones: sage-2.11, sage-2.10.4 Mar 16, 2008
@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Mar 16, 2008

comment:4

Merged in Sage 2.10.4.rc0

@sagetrac-mabshoff sagetrac-mabshoff mannequin closed this as completed Mar 16, 2008
@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Mar 16, 2008

comment:5

Replying to @TimothyClemans:

I tested this, and it looks good to me.

Timothy,

for the record: Please doctest patches you review and clearly indicate what you doctested, i.e. all of Sage, some subdirectory and so on. Applying this patch causes a doctest failure, see #2538.

Cheers,

Michael

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants