Skip to content

Create a component for the topbar#462

Merged
martinRenou merged 22 commits into
masterfrom
put_appname_and_close_btn_in_topbar
May 23, 2017
Merged

Create a component for the topbar#462
martinRenou merged 22 commits into
masterfrom
put_appname_and_close_btn_in_topbar

Conversation

@martinRenou

@martinRenou martinRenou commented May 22, 2017

Copy link
Copy Markdown
Member

This component (named ApplicationLabel) is a big dropdown button showing the current application running and its icon. When clicking on this button it displays application settings containing only the stop button for now. More settings could come later.

The stop buttons has been removed from the application list view.

This PR comes with big changes on the style of the user page. The top bar is now white, application icons are now squares with border radius of 20% instead of circles, the dropdown of the resolution widget is now less large.

Closes #415
Closes #376

@codecov-io

codecov-io commented May 22, 2017

Copy link
Copy Markdown

Codecov Report

Merging #462 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #462   +/-   ##
=======================================
  Coverage   95.33%   95.33%           
=======================================
  Files          88       88           
  Lines        4077     4077           
  Branches      259      259           
=======================================
  Hits         3887     3887           
  Misses        138      138           
  Partials       52       52

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b756507...bdc8569. Read the comment docs.

@martinRenou martinRenou requested a review from stefanoborini May 22, 2017 12:56
@@ -1,15 +1,5 @@
<template>
<section class="sidebar">
<!-- Error dialog -->

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How do we report this now? I can't remember what we settled on.

@martinRenou martinRenou May 22, 2017

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now it's in ApplicationLabel.vue, and I had some tests for it in the corresponding test file.
The reason is that it's now the application label which have the stop button so it deals with a possible stopping error.

What I don't like with it is that the modal dialog is now inside a <ul> element in the ApplicationLabel template (which is weird because the modal dialog is not a <li>), I don't know how bad this is. It works, but it's weird.

A possible fix for that would be to send an event with the error message and an other view will deal with displaying this error, same for starting error.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed #467

@martinRenou martinRenou merged commit 02b91b6 into master May 23, 2017
@martinRenou martinRenou deleted the put_appname_and_close_btn_in_topbar branch May 23, 2017 16:06
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