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

Lovelace - Weather Card: Adjust margins and move text #1880

Merged
merged 6 commits into from
Oct 29, 2018
Merged

Lovelace - Weather Card: Adjust margins and move text #1880

merged 6 commits into from
Oct 29, 2018

Conversation

timmo001
Copy link
Contributor

@timmo001 timmo001 commented Oct 27, 2018

Desctiption

Adjusts margins as to seperate out sections better.

Before

image

After

image

zsarnett and others added 4 commits October 27, 2018 11:16
* Thermostat fix

* Fix for unknown operation mode

* No title for you

* I suck at removing unnecessary things

* Fixing pointless Ternary operator
* Fix hass setting on stack

* Don't set hass on pic elements if undefined

* Don't set hass on entity rows if undefined

* prefix config prop

* Pic elements set hass yoooo

* Remove interface

* Make stack config private

* Fix import

* Lint
@ghost ghost added the in progress label Oct 27, 2018
@timmo001 timmo001 changed the title Adjust margin for forecast and now text Lovelace - Weather Card: Adjust margin for forecast and now text Oct 27, 2018
@balloob
Copy link
Member

balloob commented Oct 28, 2018

I don't like it. "Rainy" seems way too much glued to the forecast now. It's actually pretty misplaced there to begin with, it feels kinda randomply placed. I wonder if we should make the header something like Rainy <small>Dark Sky</small>

@timmo001 timmo001 changed the title Lovelace - Weather Card: Adjust margin for forecast and now text WIP: Lovelace - Weather Card: Adjust margin for forecast and now text Oct 28, 2018
@timmo001 timmo001 closed this Oct 28, 2018
@ghost ghost removed the in progress label Oct 28, 2018
@timmo001 timmo001 reopened this Oct 28, 2018
@ghost ghost added the in progress label Oct 28, 2018
@timmo001
Copy link
Contributor Author

OK. I've made the change. It now looks like this:

image

@timmo001
Copy link
Contributor Author

Also reduced some other margins that were wasting space:

image

@timmo001 timmo001 changed the title WIP: Lovelace - Weather Card: Adjust margin for forecast and now text Lovelace - Weather Card: Adjust margins and move text Oct 28, 2018
@zsarnett zsarnett changed the base branch from master to dev October 29, 2018 01:33
@zsarnett
Copy link
Contributor

ah changing the base screws everything up... Will try to fix

@balloob
Copy link
Member

balloob commented Oct 29, 2018

Nah, doesn't screw it up, just shows that not all commits from master are in dev (because cherry-picked, they get slightly different commit sha's). Merging master -> dev and now it's all good.

@balloob balloob merged commit c347be6 into home-assistant:dev Oct 29, 2018
@ghost ghost removed the in progress label Oct 29, 2018
@timmo001 timmo001 deleted the patch-1 branch October 29, 2018 09:19
@github-actions github-actions bot locked and limited conversation to collaborators Jul 7, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants