Skip to content

[ZEPPELIN-2479] Fix incorrect tooltip view inside of version control menu#2306

Closed
khalidhuseynov wants to merge 1 commit intoapache:masterfrom
khalidhuseynov:fix/ZEPPELIN-2479
Closed

[ZEPPELIN-2479] Fix incorrect tooltip view inside of version control menu#2306
khalidhuseynov wants to merge 1 commit intoapache:masterfrom
khalidhuseynov:fix/ZEPPELIN-2479

Conversation

@khalidhuseynov
Copy link
Member

@khalidhuseynov khalidhuseynov commented May 2, 2017

What is this PR for?

This is to fix incorrect appearance of tooltip inside version control menu

What type of PR is it?

Improvement

Todos

  • - fix/remove tooltip

What is the Jira issue?

ZEPPELIN-2479

How should this be tested?

go to version control menu in notebook action bar of note

Screenshots (if appropriate)

before:
screen shot 2017-05-02 at 3 32 23 pm

after:
screen shot 2017-05-02 at 6 55 24 pm

Questions:

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

@1ambda
Copy link
Member

1ambda commented May 2, 2017

Good! It would be nice to use tooltip-class with tooltip-append-to-body

@khalidhuseynov khalidhuseynov force-pushed the fix/ZEPPELIN-2479 branch 2 times, most recently from a454d54 to ac4e651 Compare May 2, 2017 10:05
@khalidhuseynov
Copy link
Member Author

@1ambda yeah that's better idea, I just pushed the changes and updated image

@1ambda
Copy link
Member

1ambda commented May 4, 2017

@khalidhuseynov Could you restart the failed travis JOB?

@1ambda
Copy link
Member

1ambda commented May 4, 2017

@Leemoonsoo
Copy link
Member

Merge to master and branch-0.7 if no further discussions.

@asfgit asfgit closed this in 8e96d8b May 5, 2017
asfgit pushed a commit that referenced this pull request May 5, 2017
…menu

This is to fix incorrect appearance of tooltip inside version control menu

Improvement

* [x] - fix/remove tooltip

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

go to `version control` menu in notebook action bar of note

before:
<img width="689" alt="screen shot 2017-05-02 at 3 32 23 pm" src="https://cloud.githubusercontent.com/assets/1642088/25606467/62f0901c-2f4c-11e7-95ef-c22f1df410be.png">

after:
![screen shot 2017-05-02 at 6 55 24 pm](https://cloud.githubusercontent.com/assets/1642088/25613839/aac216a4-2f6a-11e7-8cab-a77ddb48382b.png)

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

Author: Khalid Huseynov <khalidhnv@gmail.com>

Closes #2306 from khalidhuseynov/fix/ZEPPELIN-2479 and squashes the following commits:

ac4e651 [Khalid Huseynov] add css class

(cherry picked from commit 8e96d8b)
Signed-off-by: Lee moon soo <moon@apache.org>
michelemilesi pushed a commit to icteam-spa/zeppelin that referenced this pull request May 11, 2017
…menu

### What is this PR for?
This is to fix incorrect appearance of tooltip inside version control menu

### What type of PR is it?
Improvement

### Todos
* [x] - fix/remove tooltip

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

### How should this be tested?
go to `version control` menu in notebook action bar of note

### Screenshots (if appropriate)
before:
<img width="689" alt="screen shot 2017-05-02 at 3 32 23 pm" src="https://cloud.githubusercontent.com/assets/1642088/25606467/62f0901c-2f4c-11e7-95ef-c22f1df410be.png">

after:
![screen shot 2017-05-02 at 6 55 24 pm](https://cloud.githubusercontent.com/assets/1642088/25613839/aac216a4-2f6a-11e7-8cab-a77ddb48382b.png)

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

Author: Khalid Huseynov <khalidhnv@gmail.com>

Closes apache#2306 from khalidhuseynov/fix/ZEPPELIN-2479 and squashes the following commits:

ac4e651 [Khalid Huseynov] add css class
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.

3 participants