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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

gabriel.achille’s picture

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
FileSize
514 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

Status: Needs review » Needs work

It needs to cover the other issues identified above.

DamienMcKenna’s picture

Version: 7.x-3.x-dev » 6.x-2.x-dev
Status: Needs work » Needs review
FileSize
1018 bytes

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

DamienMcKenna’s picture

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

apaderno’s picture

Status: Needs review » Closed (outdated)

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

izmeez’s picture

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.