ICEmobile
  1. ICEmobile
  2. MOBI-149

Date/time spinner popup background css class name not correctly set.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.0 Beta
    • Fix Version/s: 1.0 Final
    • Component/s: None
    • Labels:
      None
    • Environment:
      Any

      Description

      The date and time spinner components use an empty div with the CSS class name mobi-date-bg to show dark transparent background that hides page content when the spinner popup is visible.

      On first page load the div has no style class and will alter the layout of page in some circumstances because it is in the layout of the page. However when the calendar icon is press to show the popup spinner the style class "mobi-date-bg" dynamically inserted to make the dive appear dark and transparent. And when the dialog is hidden the css class name is then changed to mobi-date.

      Preferred behaviour is as follows:

      1.) Initial style class for the div should by "mobi-date-bg-inv" instead of no style class.
      2.) The style class definition is .mobi-date-bg-inv{ display:none }
      3.) When the popup is made visible the div in question should be given the css class .mobi-date-bg{...}.
      4.) When the popup is hidden the div in question should be given the css class "mobi-date-bg-inv".

      This will insure that the div will truly be invisible and not interrupt page layout. .

        Issue Links

          Activity

          Hide
          Judy Guglielmin added a comment -

          rev 27344 contains this fix plus missing png for timespinner (get resource missing error on server).

          Show
          Judy Guglielmin added a comment - rev 27344 contains this fix plus missing png for timespinner (get resource missing error on server).
          Hide
          John Roll added a comment -

          this is still not working for us (Williams). I still see that <div id="beginDate_bg"> does not have a display:none when I inspect it in Firefox.

          Show
          John Roll added a comment - this is still not working for us (Williams). I still see that <div id="beginDate_bg"> does not have a display:none when I inspect it in Firefox.
          Hide
          John Roll added a comment -

          Here is the contents of MANIFEST.MF of icefaces-mobi.jar I am using:

          Manifest-Version: 1.0
          Ant-Version: Apache Ant 1.7.1
          Created-By: 20.4-b02-402 (Apple Inc.)

          Name: org.icefaces
          Implementation-Title: ICEmobile-Faces Mobile Components
          Implementation-Version: 1.0.0.A-0216
          Build-Number: $

          {build.number}

          Build-Time: 2012-59-16 10:59:21
          Implementation-Vendor: ICEsoft Technologies, Inc.

          Show
          John Roll added a comment - Here is the contents of MANIFEST.MF of icefaces-mobi.jar I am using: Manifest-Version: 1.0 Ant-Version: Apache Ant 1.7.1 Created-By: 20.4-b02-402 (Apple Inc.) Name: org.icefaces Implementation-Title: ICEmobile-Faces Mobile Components Implementation-Version: 1.0.0.A-0216 Build-Number: $ {build.number} Build-Time: 2012-59-16 10:59:21 Implementation-Vendor: ICEsoft Technologies, Inc.
          Hide
          Patrick Corless added a comment -

          The render is not adding the hidden style class on first render for the blackout panel. When the component is shown and hidden the style class is properly inserted. The renderer needs to be touched up to correctly this as the default state should be hidden.

          Show
          Patrick Corless added a comment - The render is not adding the hidden style class on first render for the blackout panel. When the component is shown and hidden the style class is properly inserted. The renderer needs to be touched up to correctly this as the default state should be hidden.
          Hide
          Patrick Corless added a comment -

          Updated date spinner renderer to correct the class name issue. Will have to check date spinner as well as the other panel type components.

          Show
          Patrick Corless added a comment - Updated date spinner renderer to correct the class name issue. Will have to check date spinner as well as the other panel type components.

            People

            • Assignee:
              Judy Guglielmin
              Reporter:
              Patrick Corless
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: