Closed bear454 closed 4 years ago
looks great :heart:
Merging #157 into master will not change coverage. The diff coverage is
100.00%
.
@@ Coverage Diff @@
## master #157 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 26 26
Lines 706 700 -6
=========================================
- Hits 706 700 -6
Impacted Files | Coverage Δ | |
---|---|---|
app/helpers/application_helper.rb | 100.00% <100.00%> (ø) |
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 5152059...36f139d. Read the comment docs.
@arbulu89 can you give this a once-over?
From d47293b:
Updated buttons from be81622:
WRT the width, you need to address that with the EOS maintainers; I'm only implementing the guidance:
https://suse.eosdesignsystem.com/layout
I will be doing something about the dumb jumbotron though, so that may help.
Rebased on master.
Results of dcbb4d7:
Hi @bear454 ,
If the width of the screen comes with the EOS branding, I guess we can live with it.
Besides that, except the planning in progress
gif image is not displayed yet, everything
Hi @bear454 , If the width of the screen comes with the EOS branding, I guess we can live with it. Besides that, except the planning
in progress
gif image is not displayed yet, everything
Since I'm going to be working in the plan page within the week, I think I'll just note that for now. Thanks for your help!
Hey @bear454 , is it okay if I take some screenshots from this PR to showcase Blue Horizon as a success story in EOS?
Btw, I want to say that you have done an amazing job implementing the brand. It looks outstanding.