css:validate-property-value - should not complain about 'min-height: auto'

10 views
Skip to first unread message

oliver...@mindsatwork.com.br

unread,
Apr 29, 2016, 7:17:02 PM4/29/16
to SonarQube
Hi,

I came across a case in our projects where 'min-height: auto' is specified and the rule css:validate-property-value complains about it.

It seems that this is the correct usage in the standards: https://developer.mozilla.org/en/docs/Web/CSS/min-height

The property is not included in:

Is this expected?

Thanks

David Racodon

unread,
Apr 29, 2016, 8:25:17 PM4/29/16
to oliver...@mindsatwork.com.br, SonarQube
Hi Oliver,

Thanks a lot for your feedback!

You're right. All 'min-height', 'min-width', 'max-height' and 'max-width' properties were validated the same way. But this is wrong. 'min-xxx' properties should accept 'auto' and should also not accept 'none' per the different W3C specifications.

This issue (https://github.com/SonarQubeCommunity/sonar-css/issues/165) is now fixed in the master branch and will be shipped in the next version (1.9).

Feel free to test the snapshot containing this fix that can be downloaded at https://github.com/SonarQubeCommunity/sonar-css/releases/tag/1.9-RC1

Regards,


David RACODON
Freelance QA Consultant

--
You received this message because you are subscribed to the Google Groups "SonarQube" group.
To unsubscribe from this group and stop receiving emails from it, send an email to sonarqube+...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/sonarqube/d74845a8-3f65-4ebb-aa28-99b2b7c88d34%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Oliver Von Behr Kuster

unread,
May 2, 2016, 12:12:10 PM5/2/16
to David Racodon, SonarQube
Thanks David.

I'll check it out.

Best regards,
--
 
Oliver von Behr Kuster
Desenvolvimento
www.mindsatwork.com.br
 

A informação transmitida destina-se apenas à pessoa ou entidade a quem foi endereçada e pode conter informação confidencial, legalmente protegida e para conhecimento exclusivo do destinatário. Se o leitor desta advertência não for o seu destinatário, fica ciente de que sua leitura, divulgação, distribuição ou cópia é estritamente proibida. Caso a mensagem tenha sido recebida por engano, favor comunicar ao remetente e apagar o texto do computador. // The information transmitted is intended only to the person or entity to whom it is addressed and may contain confidential information protected by law and for the exclusive knowledge of the recipient. If the reader of this warning is not your recipient is aware that his reading, dissemination, distribution or copying is strictly prohibited. If the message has been received by mistake, please notify the sender and delete the text from the computer. 
Reply all
Reply to author
Forward
0 new messages