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

Move app theming from IndexView to AppView #1479

Merged
merged 1 commit into from Sep 14, 2017

Conversation

scottnonnenberg
Copy link
Contributor

This change applies theming one level higher in the DOM so it affects the debug log popup. It's now in the overall body tag instead of its first child div.

Apply theming one level higher so it applies to the debug log.

// FREEBIE
@scottnonnenberg
Copy link
Contributor Author

I can't official approve this since I submitted the PR, but the changes are all from Lilia, so here are my comments:

Clean. Far cleaner than my solution, with the theme class added on body as well as the first div. Good to see that the CSS changes required to make it work were pretty minimal. :0)

@scottnonnenberg scottnonnenberg merged commit e2bb40d into electron Sep 14, 2017
@scottnonnenberg scottnonnenberg deleted the style-debug-log-lilia branch September 14, 2017 22:03
scottnonnenberg added a commit that referenced this pull request Sep 15, 2017
Apply theming one level higher so it applies to the debug log.

// FREEBIE
scottnonnenberg added a commit that referenced this pull request Sep 15, 2017
Update to Emojiset 4.0 (#1482)

Fix unthemed Submit Debug Log dialog (#1479)

Additional logging to help track down socket disconnect problems
(#1469)

Dismiss loading screen if we have network problems (#1466)

Setup:
  On network error during setup, show error screen instead of hanging
  (#1472)
  Allow user to setup new install with pre-profiles mobile device
  (#1470)

Ensure that we stay pinned to Windows taskbar on upgrade (#1473)
Ensure we are in the right category on Linux (#1473)
Ensure that we Heavye the right icon in the GNOME taskbar (#1473)

Transifex config specific to electron, updated translations (#1478)

CI:
  Bypass often-corrupt node_modules/.bin/grunt.cmd on windows (#1481)
  AppVeyor: Reduce build output due to xcopy of all emoji files
  (c2c9cd0)

FREEBIE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants