MioVisman / FluxBB_by_Visman

My modification of FluxBB
GNU General Public License v2.0
81 stars 14 forks source link

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

Open Dwedit opened 1 year ago

Dwedit commented 1 year ago

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.

MioVisman commented 1 year ago

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

Dwedit commented 1 year ago

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".

MioVisman commented 1 year ago

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

Dwedit commented 1 year ago

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;
}
MioVisman commented 1 year ago

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

This is not true. I added this rule:

#qjump > div > label, #qjump2 > div > label {
    display: inline-block;
}
MioVisman commented 1 year ago

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