Details
-
Type: Bug
-
Status: Closed
-
Priority: Minor
-
Resolution: Fixed
-
Affects Version/s: EE-3.3.0.GA_P01, 4.0.BETA
-
Fix Version/s: EE-3.3.0.GA_P02, 4.0
-
Component/s: ACE-Components
-
Labels:None
-
Environment:HideICEfaces trunk rev# 36979 / 37828
EE-3.3.0.GA Maintenance Branch
Server: Tomcat7
Browser: all
Test app: http://server.ice:8888/svn/repo/qa/trunk/Regression-Icefaces2/Sparkle/Nightly/dateTimeEntryShowICEfaces trunk rev# 36979 / 37828 EE-3.3.0.GA Maintenance Branch Server: Tomcat7 Browser: all Test app: http://server.ice:8888/svn/repo/qa/trunk/Regression-Icefaces2/Sparkle/Nightly/dateTimeEntry
-
Assignee Priority:P2
Description
Component styling issues are visible in the ACE regression tests for ace:dateTimeEntry (FF23, IE10, Chrome29).
These issues cannot be reproduced in showcase.
1) when multiple dateTimeEntry pages are displayed (popup or inline component) - screen shot 1.png.
2) when the dateTimeEntry component is displayed inline for a single month (pages=1) - see attached 2.png.
Steps to reproduce:
- build and deploy the dateTimeEntry test application located here: http://server.ice:8888/svn/repo/qa/trunk/Regression-Icefaces2/Sparkle/Nightly/dateTimeEntry
- issue1) can be reproduced by loading in any browser http://localhost:8080/dateTimeEntry/calendarAttribute.jsf and opening the popup calendar on the demo page, without changing any of the attributes.
- issue2) can be seen by loading in any browser: http://localhost:8080/dateTimeEntry/calendarOnly.jsf
Last revision number where both issues pass is trunk rev# 36978.
On rev# 36979 only issue 1) dateTimeEntry with multiple pages - fails; issue 2) passes.
On rev# 37062 both issues 1) and 2) fail, but the wrong styling is different from what it is now with the current revision #
These issues cannot be reproduced in showcase.
1) when multiple dateTimeEntry pages are displayed (popup or inline component) - screen shot 1.png.
2) when the dateTimeEntry component is displayed inline for a single month (pages=1) - see attached 2.png.
Steps to reproduce:
- build and deploy the dateTimeEntry test application located here: http://server.ice:8888/svn/repo/qa/trunk/Regression-Icefaces2/Sparkle/Nightly/dateTimeEntry
- issue1) can be reproduced by loading in any browser http://localhost:8080/dateTimeEntry/calendarAttribute.jsf and opening the popup calendar on the demo page, without changing any of the attributes.
- issue2) can be seen by loading in any browser: http://localhost:8080/dateTimeEntry/calendarOnly.jsf
Last revision number where both issues pass is trunk rev# 36978.
On rev# 36979 only issue 1) dateTimeEntry with multiple pages - fails; issue 2) passes.
On rev# 37062 both issues 1) and 2) fail, but the wrong styling is different from what it is now with the current revision #
Activity
Carmen Cristurean
created issue -
Carmen Cristurean
made changes -
Ken Fyten
made changes -
Assignee | Arturo Zambrano [ artzambrano ] | |
Assignee Priority | P1 [ 10010 ] |
Arturo Zambrano
made changes -
Attachment | before-after.png [ 16425 ] |
Arturo Zambrano
made changes -
Status | Open [ 1 ] | Resolved [ 5 ] |
Resolution | Fixed [ 1 ] |
Repository | Revision | Date | User | Message |
ICEsoft Public SVN Repository | #38041 | Mon Sep 16 14:48:20 MDT 2013 | art.zambrano | |
Files Changed | ||||
MODIFY
/icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/jquery/ui/jquery-ui.js
MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/datetimeentry/timepicker.css MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/datetimeentry/DateTimeEntryRenderer.java MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/jquery/ui/readme.txt |
Cruz Miraback
made changes -
Resolution | Fixed [ 1 ] | |
Status | Resolved [ 5 ] | Reopened [ 4 ] |
Cruz Miraback
made changes -
Attachment | screenshot-1.jpg [ 16482 ] |
Arturo Zambrano
made changes -
Status | Reopened [ 4 ] | Resolved [ 5 ] |
Resolution | Fixed [ 1 ] |
Repository | Revision | Date | User | Message |
ICEsoft Public SVN Repository | #38060 | Tue Sep 17 12:52:23 MDT 2013 | art.zambrano | |
Files Changed | ||||
MODIFY
/icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/jquery/ui/jquery-ui.js
MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/jquery/ui/jquery-ui.css MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/datetimeentry/timepicker.css |
Liana Munroe
made changes -
Resolution | Fixed [ 1 ] | |
Status | Resolved [ 5 ] | Reopened [ 4 ] |
Liana Munroe
made changes -
Attachment | dateTimeChrome.PNG [ 16496 ] |
Arturo Zambrano
made changes -
Status | Reopened [ 4 ] | Resolved [ 5 ] |
Resolution | Fixed [ 1 ] |
Repository | Revision | Date | User | Message |
ICEsoft Public SVN Repository | #38291 | Thu Sep 26 11:46:56 MDT 2013 | art.zambrano | |
Files Changed | ||||
MODIFY
/icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/datetimeentry/timepicker.css
|
Liana Munroe
made changes -
Resolution | Fixed [ 1 ] | |
Status | Resolved [ 5 ] | Reopened [ 4 ] |
Liana Munroe
made changes -
Attachment | dateTimeEntryAlign.PNG [ 16501 ] |
Liana Munroe
made changes -
Comment | [ The latest screenshot ] |
Ken Fyten
made changes -
Priority | Major [ 3 ] | Minor [ 4 ] |
Assignee Priority | P1 [ 10010 ] | P2 [ 10011 ] |
Ken Fyten
made changes -
Fix Version/s | EE-3.3.0.GA_P02 [ 11371 ] | |
Fix Version/s | 4.0 [ 11382 ] | |
Fix Version/s | 4.0.BETA [ 10770 ] |
Ken Fyten
made changes -
Security | Private [ 10001 ] |
Liana Munroe
made changes -
Environment |
ICEfaces trunk rev# 36979 / 37828
Server: Tomcat7 Browser: all Test app: http://server.ice:8888/svn/repo/qa/trunk/Regression-Icefaces2/Sparkle/Nightly/dateTimeEntry |
ICEfaces trunk rev# 36979 / 37828
EE-3.3.0.GA Maintenance Branch Server: Tomcat7 Browser: all Test app: http://server.ice:8888/svn/repo/qa/trunk/Regression-Icefaces2/Sparkle/Nightly/dateTimeEntry |
Liana Munroe
made changes -
Affects Version/s | EE-3.3.0.GA_P01 [ 11174 ] |
Arturo Zambrano
made changes -
Status | Reopened [ 4 ] | Resolved [ 5 ] |
Resolution | Fixed [ 1 ] |
Ken Fyten
made changes -
Status | Resolved [ 5 ] | Closed [ 6 ] |
Attaching before-after screenshot.
The cause of the first issue is that revision 36979 (
ICE-9143) introduced some localisation changes, and the weekdays went from being displayed as "Su Mo Tu We Th Fr Sa" to being displayed as "Sun Mon Tue Wed Thu Fri Sat", taking more width, while the width of the container remained the same, as seen in the before-after screenshot.This issue is easily fixed with custom app styling, but this is prone to change depending on the locale. The names of the week and other labels could take more or less space. Perhaps, we could programmatically set the width of the container, based on the width of its inner elements. We could make this behaviour configurable as well.
The second issue started at revision 37048, when the fixed width of the container was removed from the CSS file, making the calendar take as much space as possible. Again, this is easily fixed at the app level, and if can also be fixed with the approach described in the previous paragraph.