After running CSS 2.1 validation with the Multi-Day calendar turned on, several minor CSS validation issues arose. In calendar_multiday.css on lines 788 and 822, "font-style: bold;" is the current selector and that needs to be updated to "font-weight: bold;". Additionally, on 376, "min-height: auto;" needs to be adjusted to validate as well.

CommentFileSizeAuthor
#3 calendar-n1373024-3.patch1018 bytesdamienmckenna
#1 calendar-css_validation_error-1373024-1.patch514 bytesgabriel.achille

Comments

gabriel.achille’s picture

gabriel.achille commented
Title: CSS 2.1 Validation Errors » CSS 3 Validation Errors
Version: 6.x-2.4 » 7.x-3.x-dev
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new calendar-css_validation_error-1373024-1.patch514 bytes

An update and a patch on this ticket:
I found that the min-height: auto; is still there in 7.x-3.x. (while font-style: bold; is not here anymore.)
This patch fix it.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Needs work

It needs to cover the other issues identified above.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Version: 7.x-3.x-dev » 6.x-2.x-dev
Status: Needs work » Needs review
StatusFileSize
new calendar-n1373024-3.patch1018 bytes

I've added the 'font-weight' fix and the 'min-height' issue for the 6.x-2.x branch.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

Sorry, gabriel.achille's patch does resolve the problem for 7.x-3.x.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Status: Needs review » Closed (outdated)

I am closing this issue, since it's for a Drupal version no longer supported.

izmeez’s picture

izmeez commented
Title: CSS 3 Validation Errors » [calendar] CSS 3 Validation Errors
Project: Calendar » Drupal 6 Long Term Support
Version: 6.x-2.x-dev »
Component: MultiDay » Code
Status: Closed (outdated) » Reviewed & tested by the community

Patch in comment #3 has been in use for a long time.