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

Move .toccolours CSS into Twinkle #1704

Closed
NovemLinguae opened this issue Jan 23, 2023 · 1 comment · Fixed by #1720
Closed

Move .toccolours CSS into Twinkle #1704

NovemLinguae opened this issue Jan 23, 2023 · 1 comment · Fixed by #1720
Labels
bug Module: morebits The morebits.js library

Comments

@NovemLinguae
Copy link
Member

Requested by Izno at https://en.wikipedia.org/wiki/Wikipedia_talk:Twinkle#toccolours_removed_in_Vector_22

.toccolours is used by Twinkle to style something in morebits.js

.toccolours works in vector but not vector-2022

The CSS for it should probably be copied into Twinkle and named .morebits-toccolours or something. Similar to what I did in a previous patch.

@NovemLinguae NovemLinguae added bug Module: morebits The morebits.js library labels Jan 23, 2023
@siddharthvp
Copy link
Member

It's used to style the Twinkle-generated content in case it could not saved to the page due to an error:

Run:

Twinkle.speedy.callback();
Morebits.status.init($(".quickform")[0]);
Morebits.status.printUserText('{{User text}}', 'shown below is your text');

The user text appears unstyled in vector-2022.

Looks like all the required CSS is:

border: 1px solid #a2a9b1;
background-color: #f8f9fa;
padding: 5px;
font-size: 95%;

siddharthvp added a commit to siddharthvp/twinkle that referenced this issue Feb 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Module: morebits The morebits.js library
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants