23 Mar
2010
23 Mar
'10
12:30 p.m.
On 03/23/2010 07:26 AM, Caleb James DeLisle wrote:
Sergiu Dumitriu wrote:
Ah, OK, didn't know that, so this change doesn't actually alter the
upgrade process.
On 03/23/2010 05:19 AM, cjdelisle (SVN) wrote:
The stylesheet is not actually part of the LiveValidation code. I just put it there and
named it LiveValidation because
it refers to the statically defined class names used in LiveValidation. Author: cjdelisle
Date: 2010-03-23 05:19:38 +0100 (Tue, 23 Mar 2010)
New Revision: 27798
Modified:
platform/web/trunk/standard/src/main/webapp/resources/uicomponents/widgets/validation/livevalidation.css
Log:
XWIKI-5038: With Toucan skin, LiveValidation borders cause text below a field to shift
when field is clicked on. - Removed field borders from stylesheet.
This change means that upgrading to a future version will be harder. So,
the question is:
Do we override the style in our own skins, causing bigger files, but
keeping it simple to upgrade in the future, or do we fully adopt
LiveValidation and hack it as much as we want, improving it even further?
Personally I vote for the second. Still I agree with you that we should make changes to
LiveValidation and just accept more difficult upgrades.
Caleb
>
>> Modified:
platform/web/trunk/standard/src/main/webapp/resources/uicomponents/widgets/validation/livevalidation.css
>> ===================================================================
>> ---
platform/web/trunk/standard/src/main/webapp/resources/uicomponents/widgets/validation/livevalidation.css 2010-03-23
00:30:42 UTC (rev 27797)
>> +++
platform/web/trunk/standard/src/main/webapp/resources/uicomponents/widgets/validation/livevalidation.css 2010-03-23
04:19:38 UTC (rev 27798)
>> @@ -10,20 +10,4 @@
>> .LV_invalid {
>> color:#CC0000;
>> }
>> -
>> -.LV_valid_field,
>> -input.LV_valid_field:hover,
>> -input.LV_valid_field:active,
>> -textarea.LV_valid_field:hover,
>> -textarea.LV_valid_field:active {
>> - border: 1px solid #00CC00;
>> -}
>> -
>> -.LV_invalid_field,
>> -input.LV_invalid_field:hover,
>> -input.LV_invalid_field:active,
>> -textarea.LV_invalid_field:hover,
>> -textarea.LV_invalid_field:active {
>> - border: 1px solid #CC0000;
>> -}
>>
--
Sergiu Dumitriu
http://purl.org/net/sergiu/