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

Add quick oql edit feature to results view page next to current gene list #2994

Merged
merged 1 commit into from
Jan 23, 2020

Conversation

alisman
Copy link
Collaborator

@alisman alisman commented Jan 17, 2020

Add a pencil button near gene list in results page which opens interface for quickly modifying the oql of the query.

Screen Shot 2020-01-30 at 5 07 05 PM

@alisman
Copy link
Collaborator Author

alisman commented Jan 17, 2020

@adamabeshouse gonna add a e2e test for this when approved by product

@jjgao
Copy link
Member

jjgao commented Jan 20, 2020

Since the edit dialog will only be visible after click edit, can we make it bigger, e.g. the same height as in Query?

image

http://bit.ly/2v7ckrz

@jjgao
Copy link
Member

jjgao commented Jan 20, 2020

Also, if I click edit and them click "Modify Query", the "quick edit" dialog should be closed.

@jjgao jjgao requested review from jjgao and tmazor January 20, 2020 16:20
@alisman alisman force-pushed the oqlEdit branch 2 times, most recently from 436e1bd to 762cf14 Compare January 22, 2020 20:17
@alisman alisman force-pushed the oqlEdit branch 2 times, most recently from d259b1a to 49afa7c Compare January 23, 2020 20:00
@alisman alisman merged commit 4e7a80e into cBioPortal:master Jan 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants