-
-
Notifications
You must be signed in to change notification settings - Fork 351
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
Resources section and donate in-app #4135
Conversation
Make it easy to use an h2 element without applying additional CSS
Make it easy to put help bubbles anywhere and have them be consistent
@pavish please review code. @ghislaineguerin please review style. @kgodey please review copy and overall end result. |
@seancolsen Any specific reason why you decided to use cards around each resource? I was triying to avoid having cards against cards when the databases are listed, it makes the UI too busy. |
To others following along: @ghislaineguerin and I have scheduled a call tomorrow to further discuss styling. |
@ghislaineguerin and I had a quick pairing session and agreed on some changes which I pushed up as 792e530. Here is the result: |
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.
Looks good to me
…esar into resources_section
79b55e0
to
04acf13
Compare
Fixes #4058
Fixes #4057
Screenshots
Home page with no databases
Home page with databases
Responsive behavior
Top-right menu
Upgrade page
Checklist
Update index.md
).develop
branch of the repositoryDeveloper Certificate of Origin
Developer Certificate of Origin