Skip to content

Conversation

@yunho0130
Copy link
Contributor

What is this PR for?

fixed 'add from URL' button and disable backdrop function.

What type of PR is it?

Bug Fix

Todos

None

What is the Jira issue?

ZEPPELIN-1550

How should this be tested?

click 'Import note' -> click 'Add from URL' -> click 'x' button -> open the dialog again -> it shows the URL page not the initial page.

Screenshots (if appropriate)

  • before
    import_note_window
  • after
    after_2

Questions:

  • Does the licenses files need update? no
  • Is there breaking changes for older versions? no
  • Does this needs documentation? no

@yunho0130 yunho0130 changed the title fixed 'add from URL' button [ZEPPELIN-1550] fixed 'add from URL' button Oct 14, 2016
@tae-jun
Copy link
Contributor

tae-jun commented Oct 15, 2016

+1

@AhyoungRyu
Copy link
Contributor

@yunho0130 Thanks for your contribution. Nice catch!
Tested and works well. LGTM 👍

@AhyoungRyu
Copy link
Contributor

Merging this if there are no more discussions :D

@minahlee
Copy link
Member

LGTM

@asfgit asfgit closed this in e5f211b Oct 17, 2016
asfgit pushed a commit that referenced this pull request Oct 17, 2016
fixed 'add from URL' button and disable backdrop function.

Bug Fix

None

[ZEPPELIN-1550](https://issues.apache.org/jira/browse/ZEPPELIN-1550)

click 'Import note' -> click 'Add from URL' -> click 'x' button -> open the dialog again -> it shows the URL page not the initial page.

 - before
![import_note_window](https://cloud.githubusercontent.com/assets/7574765/19379302/36a0fc12-922c-11e6-9018-c67893305005.gif)

 - after
![after_2](https://cloud.githubusercontent.com/assets/7574765/19379491/2c5a072a-922d-11e6-8804-55c9766b3c5c.gif)

* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no

Author: Yunho Maeng <[email protected]>

Closes #1523 from yunho0130/ZEPPELIN-1550 and squashes the following commits:

a045928 [Yunho Maeng] fixed 'add from URL' button and disable backdrop fuction when user use modal window

(cherry picked from commit e5f211b)
Signed-off-by: Mina Lee <[email protected]>

Conflicts:
	zeppelin-web/src/components/noteName-import/note-import-dialog.html
darionyaphet pushed a commit to darionyaphet/zeppelin that referenced this pull request Oct 27, 2016
### What is this PR for?

fixed 'add from URL' button and disable backdrop function.

### What type of PR is it?
Bug Fix

### Todos
None

### What is the Jira issue?
[ZEPPELIN-1550](https://issues.apache.org/jira/browse/ZEPPELIN-1550)

### How should this be tested?
click 'Import note' -> click 'Add from URL' -> click 'x' button -> open the dialog again -> it shows the URL page not the initial page.

### Screenshots (if appropriate)

 - before
![import_note_window](https://cloud.githubusercontent.com/assets/7574765/19379302/36a0fc12-922c-11e6-9018-c67893305005.gif)

 - after
![after_2](https://cloud.githubusercontent.com/assets/7574765/19379491/2c5a072a-922d-11e6-8804-55c9766b3c5c.gif)

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no

Author: Yunho Maeng <[email protected]>

Closes apache#1523 from yunho0130/ZEPPELIN-1550 and squashes the following commits:

a045928 [Yunho Maeng] fixed 'add from URL' button and disable backdrop fuction when user use modal window
pedrozatta pushed a commit to pedrozatta/zeppelin that referenced this pull request Oct 27, 2016
### What is this PR for?

fixed 'add from URL' button and disable backdrop function.

### What type of PR is it?
Bug Fix

### Todos
None

### What is the Jira issue?
[ZEPPELIN-1550](https://issues.apache.org/jira/browse/ZEPPELIN-1550)

### How should this be tested?
click 'Import note' -> click 'Add from URL' -> click 'x' button -> open the dialog again -> it shows the URL page not the initial page.

### Screenshots (if appropriate)

 - before
![import_note_window](https://cloud.githubusercontent.com/assets/7574765/19379302/36a0fc12-922c-11e6-9018-c67893305005.gif)

 - after
![after_2](https://cloud.githubusercontent.com/assets/7574765/19379491/2c5a072a-922d-11e6-8804-55c9766b3c5c.gif)

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no

Author: Yunho Maeng <[email protected]>

Closes apache#1523 from yunho0130/ZEPPELIN-1550 and squashes the following commits:

a045928 [Yunho Maeng] fixed 'add from URL' button and disable backdrop fuction when user use modal window
Sign up for free to 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.

4 participants