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

Fix ENS onepager layout #5177

Merged
merged 1 commit into from
Sep 17, 2019
Merged

Fix ENS onepager layout #5177

merged 1 commit into from
Sep 17, 2019

Conversation

sergejmueller
Copy link
Contributor

Description

The ENS subdomain onepager is a little bit broken. I will try to fix the layout.

Before

Bildschirmfoto 2019-09-12 um 08 25 30

After

Bildschirmfoto 2019-09-12 um 08 21 37

Refers/Fixes
Testing

A designer should make a visual test of the layout changes.

* Removing duppicate stylesheets (already loaded at shared/head.html)
* Removing favicon include (already loaded at shared/head.html)
* Fixing ENS page footer
@codecov
Copy link

codecov bot commented Sep 12, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #5177   +/-   ##
=======================================
  Coverage   30.86%   30.86%           
=======================================
  Files         220      220           
  Lines       17765    17765           
  Branches     2451     2451           
=======================================
  Hits         5483     5483           
  Misses      12056    12056           
  Partials      226      226

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 a6d073c...e4ac43b. Read the comment docs.

@thelostone-mc
Copy link
Member

@sergejmueller left 2 comments but otherwise, this looks solid 🙌

@thelostone-mc thelostone-mc merged commit 6e832c5 into gitcoinco:master Sep 17, 2019
@owocki
Copy link
Contributor

owocki commented Sep 20, 2019

maybe needs a re-merge? sorry if this is related to my backing out my branch

@octavioamu
Copy link
Contributor

Don't worry we did it here #5229

@sergejmueller
Copy link
Contributor Author

Looks good, thanks for merging.

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.

6 participants