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

[ZEPPELIN-2574] Improve about dialog style #2360

Closed
wants to merge 2 commits into from

Conversation

1ambda
Copy link
Member

@1ambda 1ambda commented May 21, 2017

What is this PR for?

Improve about dialog style. See the attached screenshots.

What type of PR is it?

[Improvement]

Todos

DONE

What is the Jira issue?

ZEPPELIN-2574

How should this be tested?

  1. Build: mvn clean package -DskipTests; ./bin/zeppelin-daemon.sh restart
  2. Run Zeppelin and open browser: localhost:8080
  3. Click the About Zeppelin button

image

Screenshots (if appropriate)

Before

image

After

image

Questions:

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

@1ambda 1ambda changed the title Improve about dialgo [ZEPPELIN-2429] Improve about dialog style May 21, 2017
@1ambda 1ambda changed the title [ZEPPELIN-2429] Improve about dialog style [ZEPPELIN-2574] Improve about dialog style May 21, 2017
@1ambda 1ambda closed this May 21, 2017
@1ambda 1ambda reopened this May 21, 2017
@1ambda
Copy link
Member Author

1ambda commented May 22, 2017

@soralee Could you help review this?

@soralee
Copy link
Contributor

soralee commented May 22, 2017

@1ambda Thanks for improving dialog!
I'll test it out!

@soralee
Copy link
Contributor

soralee commented May 22, 2017

Tested and LGTM!

@Leemoonsoo
Copy link
Member

LGTM and Merge to master if no further comment

@asfgit asfgit closed this in 336d86c May 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants