-
-
Notifications
You must be signed in to change notification settings - Fork 775
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
grant/detail : rebirth #4366
grant/detail : rebirth #4366
Conversation
e0f7a68
to
71a5392
Compare
- updated page as per new design - split detail into detail/info + detail/updates - updated button styling to ensure uniformity - refactor + remove deadcode
71a5392
to
8f9524d
Compare
{% else %} | ||
|
||
<a href="{% url 'grants:fund' grant.id grant.slug %}"> | ||
<button class="btn btn-gc-blue button--full no-shadow font-weight-bold py-3">{% trans "Fund this Grant" %}</button> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe we can just use .btn-lg class will increase the radius , font size, line-height and padding
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What Im doing is just using what we have unless is a component update, then I update the component.
Codecov Report
@@ Coverage Diff @@
## master #4366 +/- ##
=======================================
Coverage 30.19% 30.19%
=======================================
Files 209 209
Lines 16790 16790
Branches 2249 2249
=======================================
Hits 5070 5070
Misses 11527 11527
Partials 193 193
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #4366 +/- ##
==========================================
- Coverage 30.19% 30.16% -0.04%
==========================================
Files 209 209
Lines 16790 16790
Branches 2249 2249
==========================================
- Hits 5070 5064 -6
- Misses 11527 11533 +6
Partials 193 193
Continue to review full report at Codecov.
|
position: relative; | ||
} | ||
|
||
#CLR_banner .clr-text { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
d-inline-block
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can't do this cause it has diff behavior on a diff screen 😅
79a3595
to
4ee7c47
Compare
app/assets/v2/css/grants/detail.css
Outdated
} | ||
|
||
.summary-container { | ||
background-color: white; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bg-white
|
||
#grant-profile-tabs { | ||
text-align: center; | ||
border-bottom: none; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
border-bottom-0
text-center
ef9fa0e
to
3e46eb0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
NOTES: donation bars will be built out in a follow-up PR
Refers/Fixes
Testing
DEMO: https://embed.vidyard.com/share/cLrED9vprW2CqFCMa9RaDi?