FluxBB_by_Visman icon indicating copy to clipboard operation
FluxBB_by_Visman copied to clipboard

Css issues when using a really old punbb mod (`.pun fieldset .rbox input` and `.pun fieldset .rbox br` css rules)

Open Dwedit opened this issue 1 year ago • 6 comments

In the style .css files, there is a rule applied:

.pun fieldset .rbox input {
	margin: 0 9px 0 -25px;
	padding: 0;
	width: 16px;
	position: relative;
	vertical-align: middle;
}

But I had an issue with this. I was re-integrating some old code for the Punbb Attachment Mod 2.0, which makes use of input tags which aren't checkboxes, and places them inside of the rbox div. You get a tiny 16x16 button that's not properly labeled as "Browse...", and it is positioned in a bad location from the margin rule.

If you make the rule apply only to checkboxes by using .pun fieldset .rbox input[type="checkbox"] {, then only checkboxes get shrunken down to 16px and have their margin changed.

The other CSS issue I encountered was this one:

.pun fieldset .rbox br {
	display: none;
}

This forcefully removes the line breaks used by the attachment mod, so I had to comment this rule out.

Dwedit avatar May 20 '23 19:05 Dwedit

Thank you! Slightly changed. Added condition for type="radio" too.

MioVisman avatar May 21 '23 03:05 MioVisman

Also one other issue I encountered:

The math captcha can turn the "-" symbol into "to reduce". But that's not standard English for subtraction at all, I had no idea what it meant until I read the source code. "Reduce" is something you do to a fraction, like changing 5/10 into 1/2. I'd replace "to reduce" with "subtract".

Dwedit avatar May 24 '23 18:05 Dwedit

Ok. I use google translate to write in english. It is he who offers translation options.

MioVisman avatar May 25 '23 02:05 MioVisman

Caught another CSS bug...

.pun label {
	display: block;
	padding: 3px 0;
}

Having display: block; in there puts in a line break in between the "Jump To" combobox and the "Go" button. They are supposed to be on the same line.

I changed it to this:

.pun label :not(#qjump) :not(#qjump2) {
	display: block;
}

.pun label {
	padding: 3px 0;
}

Dwedit avatar May 30 '23 19:05 Dwedit

.pun label :not(#qjump) :not(#qjump2)

This is not true. I added this rule:

#qjump > div > label, #qjump2 > div > label {
	display: inline-block;
}

MioVisman avatar May 31 '23 04:05 MioVisman

and https://github.com/MioVisman/FluxBB_by_Visman/commit/e892b0380d1cb9bab519fafc2d7d9c90206ac01a https://github.com/MioVisman/FluxBB_by_Visman/commit/77358b32223ea83cf2ee3becd5953c4db80fd44e

MioVisman avatar May 31 '23 04:05 MioVisman