ICEfaces
  1. ICEfaces
  2. ICE-2946

selectInputData empty table cells not styleable in IE

          Details

          • Type: Improvement Improvement
          • Status: Closed
          • Priority: Minor Minor
          • Resolution: Fixed
          • Affects Version/s: 1.6
          • Fix Version/s: 1.7
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            ICEfaces 1.7

            Description

            Currently the rime theme styles for selectInputData do not look correct do to an issue with the selectInputData component renderer. Basically the renderer does not insert a hard space into the empty table cells and as a result IE will not draw borders around the table cell. This is a known IE issue and does not show up in the old theme as no cell level styles where applied.

              Activity

              Descending order - Click to sort in ascending order
              Hide
              Permalink
              Mark Collette added a comment -

              Empty date-less cells now render an &nbsp in the <td>. This had a side-effect in the code of making the title attribute show up for those cells with the day of the week. But, it was wrong, so I fixed it to show the right day of the week.

              Subversion 16256
              icefaces\component\src\com\icesoft\faces\component\selectinputdate\SelectInputDateRenderer.java

              Show
              Mark Collette added a comment - Empty date-less cells now render an &nbsp in the <td>. This had a side-effect in the code of making the title attribute show up for those cells with the day of the week. But, it was wrong, so I fixed it to show the right day of the week. Subversion 16256 icefaces\component\src\com\icesoft\faces\component\selectinputdate\SelectInputDateRenderer.java
              Hide
              Permalink
              Ken Fyten added a comment -

              Let's make sure we remember to update the TLD doc to add the new style class name.

              Show
              Ken Fyten added a comment - Let's make sure we remember to update the TLD doc to add the new style class name.

                People

                • Assignee:
                  Unassigned
                  Reporter:
                  Patrick Corless
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: