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

PF content background, header & card look & feel #1182

Merged
merged 8 commits into from Oct 15, 2019

Conversation

thomasmaas
Copy link
Member

@thomasmaas thomasmaas commented Sep 12, 2019

Screenshot 2019-09-13 00 01 04
Screenshot 2019-09-13 00 00 49
Screenshot 2019-09-13 00 00 32
Screenshot 2019-09-13 00 00 06
Screenshot 2019-09-12 23 59 46
Screenshot 2019-09-12 23 58 07
Screenshot 2019-09-12 23 56 44

Still lots of things to fix but to get an idea of the amount of work needed. This was done in 3 hours.

@thomasmaas thomasmaas self-assigned this Sep 13, 2019
dlabaj
dlabaj previously approved these changes Sep 16, 2019
@mcoker
Copy link

mcoker commented Sep 20, 2019

Hey @thomasmaas, from looking at the screenshots, this looks great overall! Not sure the intended scope of the POC, but a few bits of feedback:

  • I might just verify that the header icons are 16x16 (or 16px on the longer edge)
  • We don't support the footer at the bottom of the content area.
  • For the card box shadow, we use 0 1px 2px 0 rgba(3, 3, 3, 0.2);. Let me know if you'd like for me to look at anything in more detail.
  • Our blue link text color is #06c. Looks like the blue text here is a slightly lighter shade?

Let me know if you'd like me to look at anything in more detail.

@thomasmaas
Copy link
Member Author

Thanks @mcoker,

It will be a CSS-only pull request so footer removal will be out of scope. I believe I'm using the correct box shadow. Will look into blue and icon sizes but we are still using a system that scales with window size so there is no such thing as 16x16 px in current 3scale design.

@codecov
Copy link

codecov bot commented Oct 1, 2019

Codecov Report

Merging #1182 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1182      +/-   ##
==========================================
- Coverage   93.15%   93.13%   -0.02%     
==========================================
  Files        2511     2512       +1     
  Lines       83167    83266      +99     
==========================================
+ Hits        77471    77552      +81     
- Misses       5696     5714      +18
Impacted Files Coverage Δ
.../initializers/backport_pg_10_support_to_rails_4.rb 30% <0%> (-50%) ⬇️
config/initializers/postgres.rb 50% <0%> (-50%) ⬇️
config/initializers/access_token_authentication.rb 55.55% <0%> (-22.23%) ⬇️
app/models/application_record.rb 66.66% <0%> (-13.89%) ⬇️
app/queries/new_accounts_query.rb 71.73% <0%> (-10.87%) ⬇️
app/lib/three_scale/api/responder.rb 95.74% <0%> (-2.13%) ⬇️
lib/tasks/impersonation_admin_user.rake 90% <0%> (ø) ⬆️
spec/acceptance/api/user_spec.rb 100% <0%> (ø)
lib/tasks/multitenant/tenants.rake 67.92% <0%> (+11.32%) ⬆️

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 3b298e6...f65afa4. Read the comment docs.

@damianpm damianpm force-pushed the content-bg-and-cards-proof-of-concept branch from b0bf198 to 267b49a Compare October 1, 2019 09:12
@damianpm damianpm self-assigned this Oct 1, 2019
@damianpm damianpm force-pushed the content-bg-and-cards-proof-of-concept branch from a245b13 to 4d22818 Compare October 1, 2019 13:10
@thomasmaas
Copy link
Member Author

Invoice pages look off. Seems they need more horizontal padding/white border https://multitenant-admin.preview01.3scale.net/finance/invoices/682198789
Screenshot 2019-10-01 15 56 39

@damianpm damianpm force-pushed the content-bg-and-cards-proof-of-concept branch 3 times, most recently from 1aeab4b to cb6ceba Compare October 1, 2019 14:53
@damianpm
Copy link
Contributor

damianpm commented Oct 1, 2019

Invoice pages look off. Seems they need more horizontal padding/white border https://multitenant-admin.preview01.3scale.net/finance/invoices/682198789

@thomasmaas I've updated invoices page
invoices

@thomasmaas
Copy link
Member Author

thomasmaas commented Oct 1, 2019

@damianpm damianpm force-pushed the content-bg-and-cards-proof-of-concept branch from cb6ceba to 52a905e Compare October 1, 2019 15:16
@thomasmaas
Copy link
Member Author

thomasmaas commented Oct 1, 2019

@damianpm damianpm force-pushed the content-bg-and-cards-proof-of-concept branch 2 times, most recently from b8525e2 to cb590f9 Compare October 2, 2019 09:21
@thomasmaas thomasmaas changed the title [WIP] proof of concept for PF content background, header & card look & feel [WIP] PF content background, header & card look & feel Oct 7, 2019
@damianpm damianpm force-pushed the content-bg-and-cards-proof-of-concept branch 7 times, most recently from 56103e6 to 19d3950 Compare October 10, 2019 15:16
@damianpm damianpm force-pushed the content-bg-and-cards-proof-of-concept branch 4 times, most recently from 61970db to 93af2b4 Compare October 11, 2019 15:52
@damianpm damianpm force-pushed the content-bg-and-cards-proof-of-concept branch 2 times, most recently from 606a8c9 to a0d737f Compare October 14, 2019 09:59
@damianpm damianpm force-pushed the content-bg-and-cards-proof-of-concept branch from a0d737f to b012343 Compare October 14, 2019 10:46
@damianpm damianpm changed the title [WIP] PF content background, header & card look & feel PF content background, header & card look & feel Oct 15, 2019
josemigallas
josemigallas previously approved these changes Oct 15, 2019
@damianpm damianpm merged commit 967bdee into master Oct 15, 2019
@damianpm damianpm deleted the content-bg-and-cards-proof-of-concept branch November 15, 2019 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
5 participants