forgejo

mirror of https://codeberg.org/forgejo/forgejo synced 2025-10-19 10:50:52 +02:00

History

0ko 563291579e fix(ui/mde): fix switch height and buttons alignment (#9545 ) Followup to https://codeberg.org/forgejo/forgejo/pulls/9359. The custom height override for the switch in MDE was not taken into account in that PR. * How it used to be: constant height + variable padding for different scenarios * How it works now: there's no padding, just variable height After the change the override that was supposed to decrease the padding started increasing it from 0, making the switch too tall. Also two of the buttons on the toolbar used to have a completely unused `button` class, which detonated them after it was globally implemented, so it was removed. First time a button with this class was added in `4615891b9d`, then it was duplicated in `6dad457552`. Modals do not depend on presence of this class: `6610eb1dbf/web_src/js/features/common-global.js (L441)` Preview: B: https://codeberg.org/attachments/5d10ac6e-45b3-4cd3-8083-f446824d4310 A: https://codeberg.org/attachments/93a00f69-56a6-4c02-93d8-6080f51d273f Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/9545 Reviewed-by: Gusted <gusted@noreply.codeberg.org>	2025-10-08 17:53:50 +02:00
..
combomarkdowneditor.css	fix(ui/mde): fix switch height and buttons alignment (#9545 )	2025-10-08 17:53:50 +02:00
fileeditor.css

0ko 563291579e fix(ui/mde): fix switch height and buttons alignment (#9545 )

Followup to https://codeberg.org/forgejo/forgejo/pulls/9359.

The custom height override for the switch in MDE was not taken into account in that PR.
* How it used to be: constant height + variable padding for different scenarios
* How it works now: there's no padding, just variable height

After the change the override that was supposed to decrease the padding started increasing it from 0, making the switch too tall.

Also two of the buttons on the toolbar used to have a completely unused `button` class, which detonated them after it was globally implemented, so it was removed.

First time a button with this class was added in 4615891b9d, then it was duplicated in 6dad457552. Modals do not depend on presence of this class:
6610eb1dbf/web_src/js/features/common-global.js (L441)

Preview:
B: https://codeberg.org/attachments/5d10ac6e-45b3-4cd3-8083-f446824d4310
A: https://codeberg.org/attachments/93a00f69-56a6-4c02-93d8-6080f51d273f

Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/9545
Reviewed-by: Gusted <gusted@noreply.codeberg.org>

2025-10-08 17:53:50 +02:00

combomarkdowneditor.css fix(ui/mde): fix switch height and buttons alignment (#9545 ) 2025-10-08 17:53:50 +02:00

fileeditor.css