[jboss-jira] [JBoss JIRA] Commented: (JBPORTAL-2226) Clicking on potlet window decorator does not seem to work all the time on IE7

[Wesley Hales (JIRA)]

    [ https://jira.jboss.org/jira/browse/JBPORTAL-2226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12454846#action_12454846 ] 

Wesley Hales commented on JBPORTAL-2226:
----------------------------------------

Thanks for looking into this Chris, lets see If I can address all of your comments:

Unfortunately the changes you made do not work in IE 7 (or any browser for that matter). An easy way to test this is with any browser is to click directly on a mode button and without letting go of the mouse button, try to drag the window. If the window drags, this problem has not been fixed.

This MUST be handled in the markup because of portlet-mode-container being contained in the dnd-handle/dyna-decoration in the DynaDecorationRenderer. I have looked at this problem many times (and provided 2 different fixes, err proposals ;) and we can either choose to fix it by changing the markup (via default renderers) or let IE7 users deal with erratic mode button behavior. And as you can see in my first comment, my goal was to keep this transitional markup valid as well.

- This solution will work with existing themes. The themes are responsible for making changes that adapt to any new stuff coming from the renderer. Luckily, the change I made doesn't mess up the DOM at a top level. We simply need to move the portlet-mode-container outside of the handle. This should be an easy fix for each theme to adapt to.

 - The only other way I can think to handle this is to surgically remove/insert the mode-container via the DOM using javascript.

 - You said the "AJAX context" broke due to my change. What exactly broke? This change should be markup moving only, I didn't touch any functional pieces. But I may have missed an unclosed div or something. Regardless, we need a solution based on the current renderer system going forward. What do you propose?

- JBPORTAL-2327 was due to this change. We just need to use something better instead of DynaRenderOptions to see which title should be rendered. This is the tricky sot in the code trying not to break what customers may depend on this embedded markup that is rendered.

- "Initial rendering is broken in IE 6 at least, title and decorations render too high" I fixed the title, but the mode buttons are still a little too high.



> Clicking on potlet window decorator does not seem to work all the time on IE7
> -----------------------------------------------------------------------------
>
>                 Key: JBPORTAL-2226
>                 URL: https://jira.jboss.org/jira/browse/JBPORTAL-2226
>             Project: JBoss Portal
>          Issue Type: Bug
>      Security Level: Public(Everyone can see) 
>          Components: Portal Core Admin
>    Affects Versions: 2.6.7 Final, 2.7.0 Final
>            Reporter: Prabhat Jha
>            Assignee: Wesley Hales
>             Fix For: 2.6.8 Final, 2.7.2 Final
>
>
> Please see http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4187113#4187113

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira