TYPO3 Core

When it was committed to trunk rev. 9391 why it is still there in Rev. 9705?

nobody said this has been fixed with rev 9391 ...

Well, the HTML code should be changed that the size=1 doesn't get the option "multiselect" but only the ones that say "multiple=true" - right?

Yes, I think so.

exactly - as discussed on Skype I just didn't expect the class "multiselect" to be assigned to single selectboxes as well.

One thing though that came to my mind, Joey: if I select "size=2" and multiple=true - then we still have a min-height of 85px which I don't find pretty cool. So, I'm actually in favor of removing the 85px again.

I think these are two different issues.

The class should be mutliselect only for multiple selectboxes anyway.
The min-height can then still be removed but at least it will be possible to style single and multiple selectboxes differently.

Anybody working on this?

Hey Steffen.

I attached a patch which fixes both issues (renaming the class depending on the size, and the min-height of 85px which shouldn't be there anyway).

Could you send it to the core list?

Thanks.
Benni.

Ready to commit, according to the core list, Steffen :)

Being of for 3 hours and the RM gets nervous :D

Will commit next hour ;-)

rev. 10103