Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.0-Beta2, 2.0.0
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      jsf2.0, ICEfaces2.0, compat
    • Affects:
      Documentation (User Guide, Ref. Guide, etc.), Sample App./Tutorial

      Description

      Once progressMonitor is completed, then this component can be done in a similar fashion.

        Issue Links

          Activity

          Judy Guglielmin created issue -
          Judy Guglielmin made changes -
          Field Original Value New Value
          Link This issue depends on ICE-4458 [ ICE-4458 ]
          Judy Guglielmin made changes -
          Salesforce Case []
          Fix Version/s 2.0 [ 10032 ]
          Ken Fyten made changes -
          Fix Version/s 2.0-Alpha1 [ 10213 ]
          Fix Version/s 2.0-Beta [ 10032 ]
          Affects Version/s 2.0-Alpha1 [ 10213 ]
          Affects Version/s 2.0-Beta [ 10032 ]
          Ken Fyten made changes -
          Fix Version/s 2.0-Alpha2 [ 10214 ]
          Fix Version/s 2.0-Alpha1 [ 10213 ]
          Hide
          Judy Guglielmin added a comment -

          direction is to use new jsf2.0 resource handling api for fileupload--changing the iframe HTML so it posts back to a ResourceHandler looking for a multipart file upload . Mircea indicates that a custom ResrouceHandler may be required.

          Also for state saving:-
          com.icesoft.faces.component.inputfile.InputFile has (old)references to facesContext.getViewNumber() from 1.8.x, which are no longer available in ICEfaces 2.0. Ted suggests that this must be referenced from the ViewState of jsf2.0 so that fileupload is associated with the correct component tree.
          There is also a reference to (old) DOMResponseWriter.getSavedNode() which no longer exists. Greg suggests that this is no longer required as jsf2.0 maintains the saved state.

          Show
          Judy Guglielmin added a comment - direction is to use new jsf2.0 resource handling api for fileupload--changing the iframe HTML so it posts back to a ResourceHandler looking for a multipart file upload . Mircea indicates that a custom ResrouceHandler may be required. Also for state saving:- com.icesoft.faces.component.inputfile.InputFile has (old)references to facesContext.getViewNumber() from 1.8.x, which are no longer available in ICEfaces 2.0. Ted suggests that this must be referenced from the ViewState of jsf2.0 so that fileupload is associated with the correct component tree. There is also a reference to (old) DOMResponseWriter.getSavedNode() which no longer exists. Greg suggests that this is no longer required as jsf2.0 maintains the saved state.
          Hide
          Judy Guglielmin added a comment -

          requires addition of commons-fileupload.jar (currently isn't in compat/lib folder)

          Show
          Judy Guglielmin added a comment - requires addition of commons-fileupload.jar (currently isn't in compat/lib folder)
          Ken Fyten made changes -
          Fix Version/s 2.0-Beta [ 10032 ]
          Fix Version/s 2.0-Alpha2 [ 10214 ]
          Ken Fyten made changes -
          Assignee Mircea Toma [ mircea.toma ]
          Ken Fyten made changes -
          Salesforce Case []
          Assignee Priority P2
          Ken Fyten made changes -
          Assignee Mircea Toma [ mircea.toma ] Mark Collette [ mark.collette ]
          Ken Fyten made changes -
          Link This issue blocks ICE-4458 [ ICE-4458 ]
          Ken Fyten made changes -
          Link This issue depends on ICE-4458 [ ICE-4458 ]
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #20216 Wed Jan 13 15:09:08 MST 2010 judy.guglielmin disable the inputFile component for now until ICE-5032 is complete
          Files Changed
          Commit graph MODIFY /icefaces/scratchpads/glimmer/compat/samples/component-showcase/web/WEB-INF/includes/content/navigation.xhtml
          Hide
          Judy Guglielmin added a comment -

          For now, the link for inputFile component has been disabled.

          Show
          Judy Guglielmin added a comment - For now, the link for inputFile component has been disabled.
          Hide
          Ken Fyten added a comment -

          Need to research this to see what's feasible using just Ajax/xHTML.

          Show
          Ken Fyten added a comment - Need to research this to see what's feasible using just Ajax/xHTML.
          Mark Collette made changes -
          Link This issue depends on ICE-2069 [ ICE-2069 ]
          Mark Collette made changes -
          Link This issue depends on ICE-1802 [ ICE-1802 ]
          Hide
          Mark Collette added a comment - - edited

          Putting code in the Filter has one major drawback: In Servlet 2.5, JARs can't have web.xml snippets, only WARs, so it would be up to applications to configure the Filter. I'd prefer a more automatic approach. So, I've been investigating using the pre-RestoreView PhaseListener to do everything that was planned for the Filter.

          The first step is to fake out the Faces-Request HTTP header, so that PartialViewContext.isAjaxRequest() will return true, on our fake-AJAX file upload POSTs. I've made the fake HttpServletRequest fake the header, so that ExternalContext will return the appropriate header, but it appears that PartialViewContext.isAjaxRequest() is caching a previous evaluation. Searching the JSF code for all direct and indirect invocations of that method show that one invocation is happening earlier than I'm intercepting it. FacesContextFactoryImpl.getFacesContext calls ExceptionHandlerFactoryImpl.getExceptionHandler() calls PartialViewContext.isAjaxRequest(). My initial solution to this was to override the ExceptionHandlerFactory, to return an ExceptionHandlerWrapper that lazily evaluates constructing the ExceptionHandler, since it's only used at the end of each Phase anyway. But, I'd still be vulnerable from any other code, over time, that could call isAjaxRequest() first. Ted told me that we override the PartialViewContextFactory, and have our own DOMPartialViewContext, where I can just add my own method to reset the isAjaxRequest() caching. This is used on ICEfaces views, and stock JSF views with delegating. So, even when testing in stock JSF, I can use it.

          Show
          Mark Collette added a comment - - edited Putting code in the Filter has one major drawback: In Servlet 2.5, JARs can't have web.xml snippets, only WARs, so it would be up to applications to configure the Filter. I'd prefer a more automatic approach. So, I've been investigating using the pre-RestoreView PhaseListener to do everything that was planned for the Filter. The first step is to fake out the Faces-Request HTTP header, so that PartialViewContext.isAjaxRequest() will return true, on our fake-AJAX file upload POSTs. I've made the fake HttpServletRequest fake the header, so that ExternalContext will return the appropriate header, but it appears that PartialViewContext.isAjaxRequest() is caching a previous evaluation. Searching the JSF code for all direct and indirect invocations of that method show that one invocation is happening earlier than I'm intercepting it. FacesContextFactoryImpl.getFacesContext calls ExceptionHandlerFactoryImpl.getExceptionHandler() calls PartialViewContext.isAjaxRequest(). My initial solution to this was to override the ExceptionHandlerFactory, to return an ExceptionHandlerWrapper that lazily evaluates constructing the ExceptionHandler, since it's only used at the end of each Phase anyway. But, I'd still be vulnerable from any other code, over time, that could call isAjaxRequest() first. Ted told me that we override the PartialViewContextFactory, and have our own DOMPartialViewContext, where I can just add my own method to reset the isAjaxRequest() caching. This is used on ICEfaces views, and stock JSF views with delegating. So, even when testing in stock JSF, I can use it.
          Ken Fyten made changes -
          Summary compat version of component-showcase requires new fileupload solution Sparkle: New fileUpload component
          Salesforce Case []
          Affects [Documentation (User Guide, Ref. Guide, etc.), Sample App./Tutorial, Compatibility/Configuration]
          Assignee Priority P2 P1
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #21455 Thu May 20 19:56:11 MDT 2010 mark.collette ICE-5032 : compat version of component-showcase requires new fileupload solution
          Files Changed
          Commit graph DEL /icefaces/scratchpads/glimmer/core/src/main/java/org/icefaces/event/EnterKeySubmit.java
          Commit graph MODIFY /icefaces/scratchpads/glimmer/core/src/main/resources/META-INF/faces-config.xml
          Commit graph DEL /icefaces/scratchpads/glimmer/core/src/main/java/org/icefaces/event/RedirectSubmit.java
          Commit graph ADD /icefaces/scratchpads/glimmer/core/src/main/java/org/icefaces/event/FormSubmit.java
          Commit graph MODIFY /icefaces/scratchpads/glimmer/core/src/main/java/org/icefaces/context/DOMPartialViewContext.java
          Commit graph ADD /icefaces/scratchpads/glimmer/core/src/main/java/org/icefaces/event/FormScriptWriter.java
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #21456 Thu May 20 19:58:48 MDT 2010 mark.collette ICE-5032 : compat version of component-showcase requires new fileupload solution
          Files Changed
          Commit graph ADD /icefaces/scratchpads/glimmer/lib/commons-fileupload.jar
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #21457 Thu May 20 20:02:32 MDT 2010 mark.collette ICE-5032 : compat version of component-showcase requires new fileupload solution
          Files Changed
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFilesMeta.java
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFilesRenderer.java
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFilesEvent.java
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFiles.java
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFilesListener.java
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFilesPhaseListener.java
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFilesLoader.java
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFilesFormSubmit.java
          Commit graph ADD /icefaces/scratchpads/sparkle/component/resources/org.icefaces.component.inputFiles
          Commit graph ADD /icefaces/scratchpads/sparkle/component/resources/org.icefaces.component.inputFiles/inputFiles.js
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #21458 Thu May 20 20:09:07 MDT 2010 mark.collette ICE-5032 : compat version of component-showcase requires new fileupload solution
          Files Changed
          Commit graph ADD /icefaces/scratchpads/sparkle/showcase/test/src/org/icefaces/inputFiles
          Commit graph MODIFY /icefaces/scratchpads/sparkle/showcase/test/web/index.html
          Commit graph ADD /icefaces/scratchpads/sparkle/showcase/test/src/org/icefaces/inputFiles/FileBean.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/showcase/test/build.xml
          Commit graph ADD /icefaces/scratchpads/sparkle/showcase/test/web/inputFiles.xhtml
          Hide
          Mark Collette added a comment -

          Committed glimmer code that sparkle depends on.
          Subversion 21455
          Subversion 21456

          Committed sparkle inputFiles first draft.
          Subversion 21457

          Commited sparkle test app changes to include inputFiles
          Subversion 21458

          Show
          Mark Collette added a comment - Committed glimmer code that sparkle depends on. Subversion 21455 Subversion 21456 Committed sparkle inputFiles first draft. Subversion 21457 Commited sparkle test app changes to include inputFiles Subversion 21458
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #21459 Fri May 21 11:37:58 MDT 2010 mark.collette ICE-5032 : compat version of component-showcase requires new fileupload solution
          Files Changed
          Commit graph MODIFY /icefaces/scratchpads/glimmer/core/src/main/java/org/icefaces/event/FormScriptWriter.java
          Hide
          Mark Collette added a comment -

          From Deryk Sinotte:
          We generally have had to call setId() specifically in these PostAddToView events because the internally tracked id of the component leads to this issue.
          So the FormScriptWriter is using writeAttribute() but this doesn't help with the internal id which is why setId needs to be called also/instead (not sure which).

          Changed FormScriptWriter so that it will always set an id on itself. If none is specified, use a generated id.

          Subversion 21459
          glimmer\core\src\main\java\org\icefaces\event\FormScriptWriter.java

          Show
          Mark Collette added a comment - From Deryk Sinotte: We generally have had to call setId() specifically in these PostAddToView events because the internally tracked id of the component leads to this issue. So the FormScriptWriter is using writeAttribute() but this doesn't help with the internal id which is why setId needs to be called also/instead (not sure which). Changed FormScriptWriter so that it will always set an id on itself. If none is specified, use a generated id. Subversion 21459 glimmer\core\src\main\java\org\icefaces\event\FormScriptWriter.java
          Mark Collette made changes -
          Link This issue depends on ICE-5769 [ ICE-5769 ]
          Ken Fyten made changes -
          Fix Version/s 2.0-Beta [ 10231 ]
          Fix Version/s 2.0-Alpha3 [ 10032 ]
          Ken Fyten made changes -
          Link This issue blocks ICE-4458 [ ICE-4458 ]
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #21748 Wed Jun 16 12:33:16 MDT 2010 mark.collette ICE-5032 : Sparkle: New fileUpload component
          Fix glimmer change
          Files Changed
          Commit graph MODIFY /icefaces/scratchpads/glimmer/core/src/main/java/org/icefaces/event/FormSubmit.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFilesFormSubmit.java
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/FormScriptWriter.java
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #21820 Wed Jun 30 12:46:21 MDT 2010 mark.collette ICE-5032 : Sparkle: New fileUpload component
          Files Changed
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFilesMeta.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFilesRenderer.java
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFilesInfo.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFilesEvent.java
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFilesCallback.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFiles.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFilesListener.java
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFilesStatus.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFilesPhaseListener.java
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFilesConfig.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFilesLoader.java
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFilesStatuses.java
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #21821 Wed Jun 30 12:57:17 MDT 2010 mark.collette ICE-5032 : Sparkle: New fileUpload component
          Test page
          Files Changed
          Commit graph MODIFY /icefaces/scratchpads/sparkle/showcase/test/src/org/icefaces/inputFiles/FileBean.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/showcase/test/web/inputFiles.xhtml
          Hide
          Mark Collette added a comment -

          Committed sparkle code for base functionality of inputFiles component.
          Subversion 21820

          TODO:

          • Change hard-coded FacesMessage strings to use proper localisations
          • Augment InputFilesStatuses to work with localised strings
          • Handle required property
          • Make info property iteration aware
          • ContentType / file extension filtering
          • Progress: HTML 5
          • Progress: HTML 4
          • Multi-file: HTML 5
          • Multi-file: HTML 4
          • Callback support
          Show
          Mark Collette added a comment - Committed sparkle code for base functionality of inputFiles component. Subversion 21820 TODO: Change hard-coded FacesMessage strings to use proper localisations Augment InputFilesStatuses to work with localised strings Handle required property Make info property iteration aware ContentType / file extension filtering Progress: HTML 5 Progress: HTML 4 Multi-file: HTML 5 Multi-file: HTML 4 Callback support
          Hide
          Mark Collette added a comment -

          Committed my test page, which runs through 12 or so scenarios of using the inputFiles component.
          Subversion 21821

          Show
          Mark Collette added a comment - Committed my test page, which runs through 12 or so scenarios of using the inputFiles component. Subversion 21821
          Ken Fyten made changes -
          Issue Type Bug [ 1 ] New Feature [ 2 ]
          Salesforce Case []
          Security Private [ 10001 ]
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #21830 Thu Jul 01 12:45:07 MDT 2010 mark.collette ICE-5032 : Sparkle: New fileUpload component
          Files Changed
          Commit graph ADD /icefaces/scratchpads/glimmer/core/src/main/java/org/icefaces/util/CoreUtils.java
          Hide
          Mark Collette added a comment -

          I missed a support file in glimmer, CoreUtils, which is derived from the similarly named class in compat.

          Subversion 21830

          Show
          Mark Collette added a comment - I missed a support file in glimmer, CoreUtils, which is derived from the similarly named class in compat. Subversion 21830
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #21850 Tue Jul 06 13:58:29 MDT 2010 mark.collette ICE-5032 : Sparkle: New fileUpload component
          Files Changed
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFilesMeta.java
          Hide
          Mark Collette added a comment -

          Added more documentation for the inputFiles properties.
          Subversion 21850

          Show
          Mark Collette added a comment - Added more documentation for the inputFiles properties. Subversion 21850
          Ken Fyten made changes -
          Fix Version/s 2.0-Beta2 [ 10242 ]
          Fix Version/s 2.0-Beta1 [ 10231 ]
          Hide
          Ken Fyten added a comment - - edited

          Must-haves for release are (in order of priority):

          • Make info property iteration aware
          • Change hard-coded FacesMessage strings to use proper localisations
          • Augment InputFilesStatuses to work with localised strings
          • Skin support (Rime + Sam)
          • Handle required property
          • ARIA support

          Next highest:

          • Progress: HTML 5
          • Progress: HTML 4

          Finally:

          • Multi-file: HTML 5
          • Multi-file: HTML 4
          • Callback support
          Show
          Ken Fyten added a comment - - edited Must-haves for release are (in order of priority): Make info property iteration aware Change hard-coded FacesMessage strings to use proper localisations Augment InputFilesStatuses to work with localised strings Skin support (Rime + Sam) Handle required property ARIA support Next highest: Progress: HTML 5 Progress: HTML 4 Finally: Multi-file: HTML 5 Multi-file: HTML 4 Callback support
          Hide
          Mark Collette added a comment -

          There's also the issue of enabling some kind of singleSubmit/partialSubmit behaviour.

          Show
          Mark Collette added a comment - There's also the issue of enabling some kind of singleSubmit/partialSubmit behaviour.
          Hide
          Mark Collette added a comment -

          The previous ice:inputFile component did not participate in form submission. It could optionally cause the side effect of submitting the form, before or after the file was uploaded. The new ice:inputFiles component does participate in form submission by default. If you use an h:commandButton to do a full form submit, then the file will upload along with the form contents. If you use an explicitly ajax form submit, like an ice:commandButton which calls javascript ice.submit, or an f:ajax h:commandButton, which calls JSF's ajax javascript, then the file will not be uploaded as the form contents are submitted. There might be a point to having a flag for the ice:commandButton to allow it to do a full form submit, so that it can upload the file.

          As well, there's on mode missing. We opt in to the form by default, and we can opt out of uploading the file by using ajax submit, but we can't currently upload the file and opt out of processing the rest of the form. As such we lack some equivalent to singleSubmit for the ice:inputFiles component. Since there is no built-in submit button on the ice:inputFiles component, it's not obvious how that could be accomplished, beyond putting the ice:inputFiles component in its own form. One use case might be the auto-upload, in which case there would be a trigger for the ice:inputFiles component to be initiating the form submission, and thus be able to direct the process to only execute the inputFiles component. Alternatively, since the InputFilesPhaseListener is responsible for setting up the whole lifecycle, the ice:inputFiles component could have some hidden input field, that would set a singleSubmit flag, which the InputFilesPhaseListener could find, and then setup the lifecycle to then only execute the ice:inputFiles component.

          Show
          Mark Collette added a comment - The previous ice:inputFile component did not participate in form submission. It could optionally cause the side effect of submitting the form, before or after the file was uploaded. The new ice:inputFiles component does participate in form submission by default. If you use an h:commandButton to do a full form submit, then the file will upload along with the form contents. If you use an explicitly ajax form submit, like an ice:commandButton which calls javascript ice.submit, or an f:ajax h:commandButton, which calls JSF's ajax javascript, then the file will not be uploaded as the form contents are submitted. There might be a point to having a flag for the ice:commandButton to allow it to do a full form submit, so that it can upload the file. As well, there's on mode missing. We opt in to the form by default, and we can opt out of uploading the file by using ajax submit, but we can't currently upload the file and opt out of processing the rest of the form. As such we lack some equivalent to singleSubmit for the ice:inputFiles component. Since there is no built-in submit button on the ice:inputFiles component, it's not obvious how that could be accomplished, beyond putting the ice:inputFiles component in its own form. One use case might be the auto-upload, in which case there would be a trigger for the ice:inputFiles component to be initiating the form submission, and thus be able to direct the process to only execute the inputFiles component. Alternatively, since the InputFilesPhaseListener is responsible for setting up the whole lifecycle, the ice:inputFiles component could have some hidden input field, that would set a singleSubmit flag, which the InputFilesPhaseListener could find, and then setup the lifecycle to then only execute the ice:inputFiles component.
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #22423 Fri Sep 24 16:36:21 MDT 2010 mark.collette ICE-5032 : Sparkle: New fileUpload component
          Files Changed
          Commit graph MODIFY /icefaces/scratchpads/sparkle/showcase/test/src/org/icefaces/inputFiles/FileBean.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/showcase/test/web/inputFiles.xhtml
          Hide
          Mark Collette added a comment -

          Augmented the testing to cover having inputFiles components inside of dataTables, with both the info property being maintained by the component, and with a ValueExpression being maintained by the bean. Also, added some test code for faces messages.

          Subversion 22423

          Show
          Mark Collette added a comment - Augmented the testing to cover having inputFiles components inside of dataTables, with both the info property being maintained by the component, and with a ValueExpression being maintained by the bean. Also, added some test code for faces messages. Subversion 22423
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #22502 Mon Oct 04 12:33:58 MDT 2010 mark.collette ICE-5032 : Sparkle: New fileUpload component
          Files Changed
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/build.xml
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/resources/messages_nl.properties
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/resources/messages_de.properties
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/resources/messages_es.properties
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/resources/messages_fr.properties
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFilesPhaseListener.java
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/resources
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/resources/org.icefaces.component.inputFiles/inputFiles.js
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/resources/messages_ar.properties
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFilesMeta.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFilesInfo.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFilesRenderer.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFiles.java
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/utils/MessageUtils.java
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/resources/messages.properties
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFilesStatus.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFilesStatuses.java
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #22503 Mon Oct 04 12:35:21 MDT 2010 mark.collette ICE-5032 : Sparkle: New fileUpload component
          Files Changed
          Commit graph ADD /icefaces/scratchpads/sparkle/showcase/test/src/org/icefaces/demo/resources/messages_es.properties
          Commit graph ADD /icefaces/scratchpads/sparkle/showcase/test/src/org/icefaces/demo/resources/messages_de.properties
          Commit graph MODIFY /icefaces/scratchpads/sparkle/showcase/test/web/WEB-INF/faces-config.xml
          Commit graph ADD /icefaces/scratchpads/sparkle/showcase/test/src/org/icefaces/demo/resources/messages_nl.properties
          Commit graph ADD /icefaces/scratchpads/sparkle/showcase/test/src/org/icefaces/demo/resources/messages_fr.properties
          Commit graph ADD /icefaces/scratchpads/sparkle/showcase/test/src/org/icefaces/demo/resources/messages_ar.properties
          Commit graph MODIFY /icefaces/scratchpads/sparkle/showcase/test/src/org/icefaces/inputFiles/FileBean.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/showcase/test/build.xml
          Commit graph ADD /icefaces/scratchpads/sparkle/showcase/test/src/org/icefaces/demo/resources/messages.properties
          Commit graph ADD /icefaces/scratchpads/sparkle/showcase/test/src/org/icefaces/demo/resources
          Commit graph MODIFY /icefaces/scratchpads/sparkle/showcase/test/web/inputFiles.xhtml
          Hide
          Mark Collette added a comment -

          Finished the support for localised FacesMessage to display depending on the results of the file upload attempt. The ice:inputFiles component will first see if the maxTotalSizeMessage, maxFileSizeMessage, maxFileCountMessage, requiredMessage properties are specified, and if not, will then see if the application has registered itw own ResourceBundle, and has specified resources for the success or error message, and failing that, will fall back to using the built-in resource bundle that comes with the sparkle components jar.

          Added MessageUtils, which is derived from compat, but re-made, to facilitate adding FacesMessage(s) derived from resource bundles, and ones from custom MessageFormat patterns.

          Future support has been laid for custom InputFilesStatus implementors to give their own custom localised FacesMessage objects.

          From testing, noticed that maxFileCount had not been implemented, most likely because multi-file has not been implemented. Finished that, so could test the messages for it. Also fixed whichever glitches I came across when testing.

          Subversion 22502

          Show
          Mark Collette added a comment - Finished the support for localised FacesMessage to display depending on the results of the file upload attempt. The ice:inputFiles component will first see if the maxTotalSizeMessage, maxFileSizeMessage, maxFileCountMessage, requiredMessage properties are specified, and if not, will then see if the application has registered itw own ResourceBundle, and has specified resources for the success or error message, and failing that, will fall back to using the built-in resource bundle that comes with the sparkle components jar. Added MessageUtils, which is derived from compat, but re-made, to facilitate adding FacesMessage(s) derived from resource bundles, and ones from custom MessageFormat patterns. Future support has been laid for custom InputFilesStatus implementors to give their own custom localised FacesMessage objects. From testing, noticed that maxFileCount had not been implemented, most likely because multi-file has not been implemented. Finished that, so could test the messages for it. Also fixed whichever glitches I came across when testing. Subversion 22502
          Hide
          Mark Collette added a comment -

          Added form14 test cases for ice:inputFiles to sparkle component showcase application, demonstrating localised faces messages.

          Subversion 22503

          Show
          Mark Collette added a comment - Added form14 test cases for ice:inputFiles to sparkle component showcase application, demonstrating localised faces messages. Subversion 22503
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #22505 Mon Oct 04 15:28:12 MDT 2010 mark.collette ICE-5032 : Sparkle: New fileUpload component
          Files Changed
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles/InputFilesRenderer.java
          Hide
          Mark Collette added a comment -

          Wrapped the input type=file element with a div tag, for the styling to act upon. As well, when we support multi-file, then all the markup will go within that root div.

          Subversion 22505

          Show
          Mark Collette added a comment - Wrapped the input type=file element with a div tag, for the styling to act upon. As well, when we support multi-file, then all the markup will go within that root div. Subversion 22505
          Hide
          Ken Fyten added a comment -

          fileEntry component has the following noted outstanding tasks/issues:

          Component Name:

          • Rename/repackage component to "FileEntry", incl. comp. name, tag name ("fileEntry"), all classnames, componentFamily, etc.

          Attributes:

          • Following attributes are missing:
          • validator
          • validatorMessage
          • tabIndex
          • disabled
          • binding
          • Following attribute names should be changed:
          • info (too general), suggest uploadStatus or uploadInfo
          • inputFilesListener -> uploadListener (aligns with uploadStatus)

          Documentation:

          • Component level descriptive doc missing from TLD/taglib doc. This can be brief summation with link to the Wiki doc page.
          • The following attributes have insufficient doc:
          • info (more detail required)
          • maxFileCount (more context - max for a single upload operation)
          • Wiki user doc:
          • No user-doc page at all yet.
          • Should mimic the general contents and compleness from http://developer.yahoo.com/yui/autocomplete/, for example.
          • General config and usage should be covered, typical use-cases illustrated and described.
          Show
          Ken Fyten added a comment - fileEntry component has the following noted outstanding tasks/issues: Component Name: Rename/repackage component to "FileEntry", incl. comp. name, tag name ("fileEntry"), all classnames, componentFamily, etc. Attributes: Following attributes are missing: validator validatorMessage tabIndex disabled binding Following attribute names should be changed: info (too general), suggest uploadStatus or uploadInfo inputFilesListener -> uploadListener (aligns with uploadStatus) Documentation: Component level descriptive doc missing from TLD/taglib doc. This can be brief summation with link to the Wiki doc page. The following attributes have insufficient doc: info (more detail required) maxFileCount (more context - max for a single upload operation) Wiki user doc: No user-doc page at all yet. Should mimic the general contents and compleness from http://developer.yahoo.com/yui/autocomplete/ , for example. General config and usage should be covered, typical use-cases illustrated and described.
          Hide
          Mark Collette added a comment -

          The InputFilesMeta should follow the dateTimeEntry pattern:

          @Component(tagName = "dateTimeEntry",
                 componentClass = "org.icefaces.component.datetimeentry.DateTimeEntry",
                 rendererClass = "org.icefaces.component.datetimeentry.DateTimeEntryRenderer",
                 componentFamily = "org.icefaces.Input",
                 componentType = "org.icefaces.DateTimeEntry",
                 rendererType = "org.icefaces.DateTimeEntry",
                 extendsClass = "javax.faces.component.UIInput",
                 generatedClass = "org.icefaces.component.datetimeentry.DateTimeEntryBase"
          )

          The binding property is now auto-inherited.

          Show
          Mark Collette added a comment - The InputFilesMeta should follow the dateTimeEntry pattern: @Component(tagName = "dateTimeEntry",        componentClass = "org.icefaces.component.datetimeentry.DateTimeEntry",        rendererClass = "org.icefaces.component.datetimeentry.DateTimeEntryRenderer",        componentFamily = "org.icefaces.Input",        componentType = "org.icefaces.DateTimeEntry",        rendererType = "org.icefaces.DateTimeEntry",        extendsClass = "javax.faces.component.UIInput",        generatedClass = "org.icefaces.component.datetimeentry.DateTimeEntryBase" ) The binding property is now auto-inherited.
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #22660 Mon Oct 18 11:25:34 MDT 2010 mark.collette ICE-5032 : compat version of component-showcase requires new fileupload solution
          Files Changed
          Commit graph ADD /icefaces/scratchpads/sparkle/showcase/test/web/fileEntry.xhtml
          Commit graph MODIFY /icefaces/scratchpads/sparkle/showcase/test/web/index.html
          Commit graph DEL /icefaces/scratchpads/sparkle/showcase/test/web/inputFiles.xhtml
          Commit graph DEL /icefaces/scratchpads/sparkle/showcase/test/web/inputFiles2.xhtml
          Commit graph ADD /icefaces/scratchpads/sparkle/showcase/test/web/fileEntry2.xhtml
          Hide
          Mark Collette added a comment - - edited

          Did the rename process of inputFiles -> fileEntry, which involved repackaging, renaming classes, renaming javascript and CSS files, and renaming CSS selectors. IDE refactoring helped a lot, but still had to follow up with directory searches of "inputfiles" (case insensitive) to fully rename everything.

          Subversion 22660
          Subversion 22667

          Show
          Mark Collette added a comment - - edited Did the rename process of inputFiles -> fileEntry, which involved repackaging, renaming classes, renaming javascript and CSS files, and renaming CSS selectors. IDE refactoring helped a lot, but still had to follow up with directory searches of "inputfiles" (case insensitive) to fully rename everything. Subversion 22660 Subversion 22667
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #22667 Mon Oct 18 13:52:51 MDT 2010 mark.collette ICE-5032 : compat version of component-showcase requires new fileupload solution
          Files Changed
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/build.xml
          Commit graph MODIFY /icefaces/scratchpads/sparkle/showcase/test/src/org/icefaces/demo/resources/messages_fr.properties
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/FileEntryMeta.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/resources/messages_es.properties
          Commit graph DEL /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/InputFilesEvent.java
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/FileEntryRenderer.java
          Commit graph DEL /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/InputFilesFormSubmit.java
          Commit graph DEL /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/InputFilesListener.java
          Commit graph DEL /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/InputFilesMeta.java
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/FileEntryInfo.java
          Commit graph DEL /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/inputFiles
          Commit graph ADD /icefaces/scratchpads/sparkle/showcase/test/src/org/icefaces/fileentry
          Commit graph DEL /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/InputFilesPhaseListener.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/showcase/test/src/org/icefaces/demo/resources/messages_es.properties
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/resources/messages_nl.properties
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/resources/messages_de.properties
          Commit graph DEL /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/InputFilesStatuses.java
          Commit graph DEL /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/InputFiles.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/resources/messages_fr.properties
          Commit graph ADD /icefaces/scratchpads/sparkle/component/resources/org.icefaces.component.fileentry
          Commit graph DEL /icefaces/scratchpads/sparkle/component/resources/org.icefaces.component.fileentry/inputFiles.js
          Commit graph DEL /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/InputFilesCallback.java
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/FileEntryCallback.java
          Commit graph DEL /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/InputFilesStatus.java
          Commit graph DEL /icefaces/scratchpads/sparkle/showcase/test/src/org/icefaces/inputFiles
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/resources/messages.properties
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/skins/sam/fileEntry.css
          Commit graph DEL /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/InputFilesRenderer.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/showcase/test/src/org/icefaces/fileentry/FileBean.java
          Commit graph DEL /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/InputFilesLoader.java
          Commit graph DEL /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/InputFilesConfig.java
          Commit graph ADD /icefaces/scratchpads/sparkle/component/resources/org.icefaces.component.fileentry/fileEntry.js
          Commit graph DEL /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/skins/sam/inputfiles.css
          Commit graph DEL /icefaces/scratchpads/sparkle/component/resources/org.icefaces.component.inputFiles
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/FileEntryLoader.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/showcase/test/src/org/icefaces/demo/resources/messages_de.properties
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/FileEntry.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/showcase/test/src/org/icefaces/demo/resources/messages_ar.properties
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/FileEntryStatus.java
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/FileEntryEvent.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/showcase/test/src/org/icefaces/demo/resources/messages.properties
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/FileEntryStatuses.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/showcase/test/src/org/icefaces/demo/resources/messages_nl.properties
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/skins/rime/fileEntry.css
          Commit graph DEL /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/InputFilesInfo.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/FormScriptWriter.java
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/FileEntryPhaseListener.java
          Commit graph DEL /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/skins/rime/inputfiles.css
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/FileEntryFormSubmit.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/resources/messages_ar.properties
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/FileEntryConfig.java
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #22692 Tue Oct 19 19:09:23 MDT 2010 mark.collette ICE-5032 : compat version of component-showcase requires new fileupload solution
          Files Changed
          Commit graph MODIFY /icefaces/scratchpads/sparkle/showcase/test/web/fileEntry.xhtml
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/FileEntryMeta.java
          Commit graph DEL /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/FileEntryInfo.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/skins/rime/fileEntry.css
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/FileEntry.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/skins/sam/fileEntry.css
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/FileEntryStatus.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/FileEntryStatuses.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/FileEntryRenderer.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/FileEntryPhaseListener.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/showcase/test/src/org/icefaces/fileentry/FileBean.java
          Commit graph ADD /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/FileEntryResults.java
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/utils/Utils.java
          Hide
          Mark Collette added a comment - - edited

          Added disabled, tabindex, style, styleClass pass-through attributes. Made some utility methods for concatenating the styleClasses. Renamed FileEntryInfo and the info property to FileEntryResults and the results property. Better documented results and maxFileCount. Added testing code for the new properties. Updated the rime and sam skins to describe the disabled style as ice-file-entry-disabled, but the renderer is not outputting that, as we're waiting for a pan sparkle naming strategy for disabled styling.

          Subversion 22692

          Small tweak to the @Component and @ResourceDependencies annotations in FileEntryMeta.

          Subversion 22693

          Show
          Mark Collette added a comment - - edited Added disabled, tabindex, style, styleClass pass-through attributes. Made some utility methods for concatenating the styleClasses. Renamed FileEntryInfo and the info property to FileEntryResults and the results property. Better documented results and maxFileCount. Added testing code for the new properties. Updated the rime and sam skins to describe the disabled style as ice-file-entry-disabled, but the renderer is not outputting that, as we're waiting for a pan sparkle naming strategy for disabled styling. Subversion 22692 Small tweak to the @Component and @ResourceDependencies annotations in FileEntryMeta. Subversion 22693
          Hide
          Mark Collette added a comment -

          Outstanding features are:

          • Disabled styling
          • Handle required property
          • Handle singleSubmit property
          • Add validator, validatorMessage properties
          • Progress: HTML 4
          • Multi-file: HTML 4 user interface and PhaseListener
          • Multi-file: HTML 5 user interface
          • Callback support

          Not doing:

          Progress via HTML 5 : This turned out to be a non-feature, since the progress is not on the input element, but just in XmlHttpRequest, and right now we're not supporting HTML XmlHttpRequest. So, we're only doing HTML 4 (and above) progress.

          ARIA support : There is no ARIA input type=file widget. There is no role. We could in theory still fulfill the basic roletype requirements. Also, when we add the interface for multiple files, those elements could have ARIA support.

          Show
          Mark Collette added a comment - Outstanding features are: Disabled styling Handle required property Handle singleSubmit property Add validator, validatorMessage properties Progress: HTML 4 Multi-file: HTML 4 user interface and PhaseListener Multi-file: HTML 5 user interface Callback support Not doing: Progress via HTML 5 : This turned out to be a non-feature, since the progress is not on the input element, but just in XmlHttpRequest, and right now we're not supporting HTML XmlHttpRequest. So, we're only doing HTML 4 (and above) progress. ARIA support : There is no ARIA input type=file widget. There is no role. We could in theory still fulfill the basic roletype requirements. Also, when we add the interface for multiple files, those elements could have ARIA support.
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #22693 Tue Oct 19 22:20:01 MDT 2010 mark.collette ICE-5032 : compat version of component-showcase requires new fileupload solution
          Files Changed
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/FileEntryMeta.java
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #22720 Wed Oct 20 23:38:46 MDT 2010 mark.collette ICE-5032 : compat version of component-showcase requires new fileupload solution
          Files Changed
          Commit graph MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/fileentry/FileEntryMeta.java
          Hide
          Mark Collette added a comment -

          Added @Component tlddoc.

          Subversion 22720

          Show
          Mark Collette added a comment - Added @Component tlddoc. Subversion 22720
          Ken Fyten made changes -
          Summary Sparkle: New fileUpload component New FileEntry ACE component
          Salesforce Case []
          Component/s ACE-Components [ 10050 ]
          Component/s Components [ 10012 ]
          Affects [Compatibility/Configuration, Documentation (User Guide, Ref. Guide, etc.), Sample App./Tutorial] [Documentation (User Guide, Ref. Guide, etc.), Sample App./Tutorial]
          Assignee Priority P1
          Affects Version/s 2.0-Alpha1 [ 10213 ]
          Ken Fyten made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #22960 Wed Oct 27 21:21:32 MDT 2010 mark.collette ICE-5032 : compat version of component-showcase requires new fileupload solution
          Files Changed
          Commit graph MODIFY /icefaces2/trunk/icefaces/ace/component/resources/org.icefaces.component.fileentry/fileEntry.js
          Hide
          Ken Fyten added a comment -

          Re-opened due to IE8 JS error issue, and IE7 styling issue.

          Show
          Ken Fyten added a comment - Re-opened due to IE8 JS error issue, and IE7 styling issue.
          Ken Fyten made changes -
          Resolution Fixed [ 1 ]
          Status Resolved [ 5 ] Reopened [ 4 ]
          Assignee Priority P1
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #22961 Wed Oct 27 22:22:54 MDT 2010 mark.collette ICE-5032 : compat version of component-showcase requires new fileupload solution
          Files Changed
          Commit graph MODIFY /icefaces2/trunk/icefaces/ace/component/src/org/icefaces/component/fileentry/skins/rime/fileEntry.css
          Commit graph MODIFY /icefaces2/trunk/icefaces/ace/component/src/org/icefaces/component/fileentry/skins/sam/fileEntry.css
          Hide
          Mark Collette added a comment -

          IE8 was getting a javascript exception, since it added support for some of the DOM features that Firefox and WebKit support, and was going down the non-IE code path, which didn't work because it still didn't support some other features. Added more precise feature detecting.

          Subversion 22960

          Show
          Mark Collette added a comment - IE8 was getting a javascript exception, since it added support for some of the DOM features that Firefox and WebKit support, and was going down the non-IE code path, which didn't work because it still didn't support some other features. Added more precise feature detecting. Subversion 22960
          Hide
          Mark Collette added a comment -

          Fixed IE7 alignment issue for fileEntry by removing overflow:hidden styling from the themes. Then noticed that rime style class was "rime", instead of "ice-skin-rime", as it was in other files, and modified it to match.

          Subversion 22961

          Show
          Mark Collette added a comment - Fixed IE7 alignment issue for fileEntry by removing overflow:hidden styling from the themes. Then noticed that rime style class was "rime", instead of "ice-skin-rime", as it was in other files, and modified it to match. Subversion 22961
          Ken Fyten made changes -
          Status Reopened [ 4 ] Resolved [ 5 ]
          Assignee Priority P1
          Resolution Fixed [ 1 ]
          Ken Fyten made changes -
          Fix Version/s 2.0.0 [ 10230 ]
          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              Mark Collette
              Reporter:
              Judy Guglielmin
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: