Skip to content

[ZEPPELIN-1552] Search button goes to next line when display's width shortens. - #1525

Closed
hyonzin wants to merge 3 commits into
apache:masterfrom
hyonzin:ZEPPELIN-1552
Closed

[ZEPPELIN-1552] Search button goes to next line when display's width shortens.#1525
hyonzin wants to merge 3 commits into
apache:masterfrom
hyonzin:ZEPPELIN-1552

Conversation

@hyonzin

Copy link
Copy Markdown
Contributor

What is this PR for?

In document page(http://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/), the search button goes to next line when display's width shortens.

What type of PR is it?

[Improvement]

What is the Jira issue?

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

Screenshots (if appropriate)

2016-10-15 2 04 00

when (768px ≤ width < 992px) ### Questions: - Does the licenses files need update? No - Is there breaking changes for older versions? No - Does this needs documentation? No

@tae-jun

Copy link
Copy Markdown
Contributor

LGTM

@AhyoungRyu

AhyoungRyu commented Oct 15, 2016

Copy link
Copy Markdown
Contributor

@hyonzin Thanks for your contribution and good catch :)
Actually I added this search feature in #1266 so it's my bad.. Appreciate for taking care of this.

Just tested and it seems not working as expected in my 13 inch screen. (FYI 13inch -> about 1248px)

before

So I worked around a bit to fix this. You can check it in my branch. With my change it will be like this.

after

If you have time, please check this and let me know how it is! :D

@hyonzin

Copy link
Copy Markdown
ContributorAuthor

@AhyoungRyu Thank you for your reply and good suggestion. I checked that in your branch and It seems to work very well!

@AhyoungRyu

Copy link
Copy Markdown
Contributor

@hyonzin Thx for your prompt response!
Seems the CI is not happy now with below reason

No output has been received in the last 10 minutes, this potentially indicates a stalled build or something wrong with the build itself.
The build has been terminated

LGTM 👍

@hyonzinhyonzin closed this Oct 15, 2016
@hyonzinhyonzin reopened this Oct 15, 2016
@tae-jun

Copy link
Copy Markdown
Contributor

CI is green light!

Merge please @AhyoungRyu ! 😄

@AhyoungRyu

Copy link
Copy Markdown
Contributor

Yeah true! Thanks for your quick verification @tae-jun :)
And thanks again @hyonzin!

Merge if there are no more discussions.

@hyonzin

Copy link
Copy Markdown
ContributorAuthor

@AhyoungRyu@tae-jun Thanks for you verification 😀

darionyaphet pushed a commit to darionyaphet/zeppelin that referenced this pull request Oct 27, 2016
…shortens.
### What is this PR for?
In document page(http://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/), the search button goes to next line when display's width shortens.
### What type of PR is it?
[Improvement]
### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-1552
### Screenshots (if appropriate)
<img width="927" alt="2016-10-15 2 04 00" src="https://cloud.githubusercontent.com/assets/6567102/19398321/0d76a1c4-9287-11e6-86e0-9f120c00b143.png">
when (768px ≤ width < 992px)
### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No
Author: hyonzin <hyeonjin507@gmail.com>
Author: AhyoungRyu <ahyoungryu@apache.org>
Author: 정현진 <hyeonjin507@gmail.com>
Closesapache#1525 from hyonzin/ZEPPELIN-1552 and squashes the following commits:
f1d97ef [AhyoungRyu] Fix search icon's placement in small browser
244992d [정현진] Straighten wrong term
97d1bdf [hyonzin] Fix search link button to not goes next line when display's width shortens
pedrozatta pushed a commit to pedrozatta/zeppelin that referenced this pull request Oct 27, 2016
…shortens.
### What is this PR for?
In document page(http://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/), the search button goes to next line when display's width shortens.
### What type of PR is it?
[Improvement]
### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-1552
### Screenshots (if appropriate)
<img width="927" alt="2016-10-15 2 04 00" src="https://cloud.githubusercontent.com/assets/6567102/19398321/0d76a1c4-9287-11e6-86e0-9f120c00b143.png">
when (768px ≤ width < 992px)
### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No
Author: hyonzin <hyeonjin507@gmail.com>
Author: AhyoungRyu <ahyoungryu@apache.org>
Author: 정현진 <hyeonjin507@gmail.com>
Closesapache#1525 from hyonzin/ZEPPELIN-1552 and squashes the following commits:
f1d97ef [AhyoungRyu] Fix search icon's placement in small browser
244992d [정현진] Straighten wrong term
97d1bdf [hyonzin] Fix search link button to not goes next line when display's width shortens
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.

3 participants

@hyonzin@tae-jun@AhyoungRyu