-
-
Notifications
You must be signed in to change notification settings - Fork 277
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
MBS-13062: Implementing the footer redesign #2926
MBS-13062: Implementing the footer redesign #2926
Conversation
Can one of the admins verify this patch? |
8e4c021
to
0a92f16
Compare
@brainzbot, add to whitelist |
1d0e331
to
c4eee04
Compare
So, the failing test is now passing: But a new error arises... which I don't know how to solve:
@reosarevok maybe you can give me an insight about this? Thanks! |
52098e1
to
7897763
Compare
- Created a file with the raw input data used to build the footer - Most parts of the footer were moved to their own components
- Row-cols is used to define the layout on different breakpoints - Added margins on breakpoints < xl - Fixed margins in first block
- Added text about MusicBrainz - Change channel's list line-height - Added gray style - Removed <bold> from links
7897763
to
522ebd3
Compare
@brainzbot, retest this please |
I'm not sure tbh. I hope it's not something stupid like [] in the test title breaking something? :) |
Merging to the footer revamp PR for now because we need to resolve conflicts and it makes sense to resolve them at one place |
b52778b
into
metabrainz:akshat/footer-revamp
I'm updating the branch that @akshaaatt created, as it was in #2439, with some changes & fixes:
MBS-4555.json5
was failing because it couldn't find an element. Fixed it creating a newid
The new looks: