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

Fix anon edit warning message styling #266

Merged
merged 3 commits into from
Aug 24, 2022

Conversation

chukarave
Copy link
Contributor

they were in the wrong component!

Bug: T312652

they were in the wrong component!

Bug: T312652
@@ -26,5 +26,7 @@ const config = useConfig();

.wbl-snl-anonymous-edit-warning {
margin: $dimension-spacing-small 0;
margin-top: 1rem;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no, these lines are now just overriding what is in the line above

Copy link
Collaborator

@micgro42 micgro42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good thanks!
Now

@import "@wmde/wikit-tokens/variables";

can be deleted, I think, and then this is ready to go :)

@chukarave chukarave merged commit 93a0128 into main Aug 24, 2022
@chukarave chukarave deleted the fix-anonymous-warning-message-margins-styling branch August 24, 2022 06:50
wmfgerrit pushed a commit to wikimedia/mediawiki-extensions-WikibaseLexeme that referenced this pull request Aug 24, 2022
This is mostly to make sure no unnecessary margins
come to be in other areas of the inforpanel due to
unsustainable :first-child use
Added built warning message styling fix from PR #266
(wmde/new-lexeme-special-page#266)

Bug: T312652
Change-Id: Ie471cc4d8033220da361a563c681d147266613e7
wmfgerrit pushed a commit to wikimedia/mediawiki-extensions that referenced this pull request Aug 24, 2022
* Update WikibaseLexeme from branch 'master'
  to b4c402f421262d0dbb02aa9b41e69e07347d0cbb
  - Lexidata: add class names and margins to infopanel elements
    
    This is mostly to make sure no unnecessary margins
    come to be in other areas of the inforpanel due to
    unsustainable :first-child use
    Added built warning message styling fix from PR #266
    (wmde/new-lexeme-special-page#266)
    
    Bug: T312652
    Change-Id: Ie471cc4d8033220da361a563c681d147266613e7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants