Skip to content

Option in the Zeppelin GUI to clear the output in a specific note fro… - #1748

Closed
PetroKostyuk wants to merge 1 commit into
apache:masterfrom
PetroKostyuk:ZEPPELIN-1503
Closed

Option in the Zeppelin GUI to clear the output in a specific note fro…#1748
PetroKostyuk wants to merge 1 commit into
apache:masterfrom
PetroKostyuk:ZEPPELIN-1503

Conversation

@PetroKostyuk

Copy link
Copy Markdown

What is this PR for?

It allowes user to clear notes input from menu. It resolves problem in issue 1503

What type of PR is it?

Feature

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-1503

How should this be tested?

Clicking on new icon should clear output of all notes in given notebook

Screenshots (if appropriate)

screenshot from 2016-12-12 21-01-45

@felixcheung

Copy link
Copy Markdown
Member

thanks for the contribution. since this is the notebook menu, it might not be very clear what that icon means or what that "button" does (for example, is it for erasing the notebook? deleting the notebook?) Do you think there is some ways or other places for this?

@PetroKostyuk

Copy link
Copy Markdown
Author

I tought it wold be enought to give it same icon as is used for clearing output in other places. Also, there is tooltip 'clear notes output' on that button, as I have seen in other places in Zeppelin. Sorry i didn't put screenshot of tooltip, i couldn't hover mouse on it same time as selecting area of screenshoot by mouse. But if you have some other idea how to make it clear, I'm opened for suggestions.

@PetroKostyuk

Copy link
Copy Markdown
Author

screenshot from 2016-12-13 14-27-28

@asfgitasfgit closed this in c38a0a0May 9, 2018
asfgit pushed a commit that referenced this pull request May 9, 2018
close#83close#86close#125close#133close#139close#146close#193close#203close#246close#262close#264close#273close#291close#299close#320close#347close#389close#413close#423close#543close#560close#658close#670close#728close#765close#777close#782close#783close#812close#822close#841close#843close#878close#884close#918close#989close#1076close#1135close#1187close#1231close#1304close#1316close#1361close#1385close#1390close#1414close#1422close#1425close#1447close#1458close#1466close#1485close#1492close#1495close#1497close#1536close#1545close#1561close#1577close#1600close#1603close#1678close#1695close#1739close#1748close#1765close#1767close#1776close#1783close#1799
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@PetroKostyuk@felixcheung