Re: recent console changes in trunk
Thanks, I modified our copy of pluto.css in trunk to comment out the label definition, until we either merge everything into our main.css or decide just to use this hack as-is. -Donald Jack wrote: I searched the keyword label in all JSP files in an older revision which is before all the accessibility patches came in, and found no occurrences of label element. So I assume it's probably safe to remove the label style in pluto.css. Anyway it's safer if we can use main.css to override pluto.css. I can check this out later on. 2008/7/15 Joe Bohn [EMAIL PROTECTED] mailto:[EMAIL PROTECTED]: YunFeng Ma wrote: I tested the console in FireFox 2.x and IE 6.x after removing label tag in pluto.css and didn't see any problem. I prefer to remove it before G v2.1.2 is out of the door. I agree we want this fixed before we ship 2.1.2. http://2.1.2. However, I'm not sure modifying pluto.css is the best way to go. All of our style sheet changes for the console have typically been restricted to main.css. I think pluto.css is pretty much a direct copy from pluto. It might be easier to maintain if we continue to keep our changes limited to main.css. Can you try fixing this by adding a new label entry in main.css without float:left? Joe smime.p7s Description: S/MIME Cryptographic Signature
Re: recent console changes in trunk
I've opened a JIRA (GERONIMO-4204) for this issue. I need to know if the float:left for labels in pluto.css is needed and where because it affects all Admin Console portlets. On Mon, Jul 14, 2008 at 3:09 PM, Shrey Banga [EMAIL PROTECTED] wrote: After rebuilding the server I too faced the same problem and on further inspection I could make out the problem was because the patch GERONIMO-4081 Accessibility issues (at revision: 675083) has surrounded text by label tags, which have a float:left property set in pluto.css which I suppose is causing the problems. I'm not sure why the float:left is needed in the first place but in case removing it isn't possible, we could create a separate css for the console and import it in the affected portlets. On Fri, Jul 11, 2008 at 6:50 PM, Joe Bohn [EMAIL PROTECTED] wrote: In addition to that quirk, there are other layout issues I'm seeing too (these are probably the ones Jarek was seeing). For example: - Server Log Viewer layout is all messed up - All of the Applications views (WARs, EARs, etc...) have the layout all messed up - Plugin portlets layout is messed up Those are just a few that I noticed clicking on the top level portlet pages. There may be other problems when you drill down into some of the pages to alternate jsps. Joe Joe Bohn wrote: I'm seeing strange things in trunk as well. Using firefox on a tomcat image I see the following prior to the banner in the console. I don't see this in safari: Skip to navigation Skip to main content The access keys for this page are: * ALT plus 1 skips to navigation. * ALT plus 2 skips to main content. The first two are links to anchors in the page. If I reload the page the extra content prior to the banner is no longer displayed and things look normal again. Joe Shiva Kumar H R wrote: I have been committing some patches in Admin Console, but most of them have been local to Plan Creator portlet. On the server that's built on my machine (which has all the commits I have done so far + trunk code as on 26-Jun), the portlets that you mention (Web App WARs, Deploy New, Plugins, Server Logs) seem to be working fine (I tested using IE 6 Firefox 3). I will update my trunk and do a clean build, and see if I face any problems. On Thu, Jul 10, 2008 at 9:59 PM, Jarek Gawor [EMAIL PROTECTED] mailto: [EMAIL PROTECTED] wrote: Hi, I just built the latest code tried to use the admin console and the layout in a few places (e.g. Web App WARs, Deploy New, Plugins, Server Logs) is screwed up. I see similar layout issues on Firefox (2.0) and IE. It does not seem like people are checking how things look in the console before committing the patches... Jarek -- Thanks, Shiva -- Shrey Banga Bachelor of Technology, III year Department of Electrical Engineering Indian Institute of Technology Roorkee -- Shrey Banga Bachelor of Technology, III year Department of Electrical Engineering Indian Institute of Technology Roorkee
Re: recent console changes in trunk
I tested the console in FireFox 2.x and IE 6.x after removing label tag in pluto.css and didn't see any problem. I prefer to remove it before G v2.1.2 is out of the door. -- Yun Feng Shrey Banga wrote: I've opened a JIRA (GERONIMO-4204) for this issue. I need to know if the float:left for labels in pluto.css is needed and where because it affects all Admin Console portlets. On Mon, Jul 14, 2008 at 3:09 PM, Shrey Banga [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: After rebuilding the server I too faced the same problem and on further inspection I could make out the problem was because the patch GERONIMO-4081 Accessibility issues (at revision: 675083) has surrounded text by label tags, which have a float:left property set in pluto.css which I suppose is causing the problems. I'm not sure why the float:left is needed in the first place but in case removing it isn't possible, we could create a separate css for the console and import it in the affected portlets. On Fri, Jul 11, 2008 at 6:50 PM, Joe Bohn [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: In addition to that quirk, there are other layout issues I'm seeing too (these are probably the ones Jarek was seeing). For example: - Server Log Viewer layout is all messed up - All of the Applications views (WARs, EARs, etc...) have the layout all messed up - Plugin portlets layout is messed up Those are just a few that I noticed clicking on the top level portlet pages. There may be other problems when you drill down into some of the pages to alternate jsps. Joe Joe Bohn wrote: I'm seeing strange things in trunk as well. Using firefox on a tomcat image I see the following prior to the banner in the console. I don't see this in safari: Skip to navigation Skip to main content The access keys for this page are: * ALT plus 1 skips to navigation. * ALT plus 2 skips to main content. The first two are links to anchors in the page. If I reload the page the extra content prior to the banner is no longer displayed and things look normal again. Joe Shiva Kumar H R wrote: I have been committing some patches in Admin Console, but most of them have been local to Plan Creator portlet. On the server that's built on my machine (which has all the commits I have done so far + trunk code as on 26-Jun), the portlets that you mention (Web App WARs, Deploy New, Plugins, Server Logs) seem to be working fine (I tested using IE 6 Firefox 3). I will update my trunk and do a clean build, and see if I face any problems. On Thu, Jul 10, 2008 at 9:59 PM, Jarek Gawor [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: Hi, I just built the latest code tried to use the admin console and the layout in a few places (e.g. Web App WARs, Deploy New, Plugins, Server Logs) is screwed up. I see similar layout issues on Firefox (2.0) and IE. It does not seem like people are checking how things look in the console before committing the patches... Jarek -- Thanks, Shiva -- Shrey Banga Bachelor of Technology, III year Department of Electrical Engineering Indian Institute of Technology Roorkee -- Shrey Banga Bachelor of Technology, III year Department of Electrical Engineering Indian Institute of Technology Roorkee
Re: recent console changes in trunk
YunFeng Ma wrote: I tested the console in FireFox 2.x and IE 6.x after removing label tag in pluto.css and didn't see any problem. I prefer to remove it before G v2.1.2 is out of the door. I agree we want this fixed before we ship 2.1.2. However, I'm not sure modifying pluto.css is the best way to go. All of our style sheet changes for the console have typically been restricted to main.css. I think pluto.css is pretty much a direct copy from pluto. It might be easier to maintain if we continue to keep our changes limited to main.css. Can you try fixing this by adding a new label entry in main.css without float:left? Joe -- Yun Feng Shrey Banga wrote: I've opened a JIRA (GERONIMO-4204) for this issue. I need to know if the float:left for labels in pluto.css is needed and where because it affects all Admin Console portlets. On Mon, Jul 14, 2008 at 3:09 PM, Shrey Banga [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: After rebuilding the server I too faced the same problem and on further inspection I could make out the problem was because the patch GERONIMO-4081 Accessibility issues (at revision: 675083) has surrounded text by label tags, which have a float:left property set in pluto.css which I suppose is causing the problems. I'm not sure why the float:left is needed in the first place but in case removing it isn't possible, we could create a separate css for the console and import it in the affected portlets. On Fri, Jul 11, 2008 at 6:50 PM, Joe Bohn [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: In addition to that quirk, there are other layout issues I'm seeing too (these are probably the ones Jarek was seeing). For example: - Server Log Viewer layout is all messed up - All of the Applications views (WARs, EARs, etc...) have the layout all messed up - Plugin portlets layout is messed up Those are just a few that I noticed clicking on the top level portlet pages. There may be other problems when you drill down into some of the pages to alternate jsps. Joe Joe Bohn wrote: I'm seeing strange things in trunk as well. Using firefox on a tomcat image I see the following prior to the banner in the console. I don't see this in safari: Skip to navigation Skip to main content The access keys for this page are: * ALT plus 1 skips to navigation. * ALT plus 2 skips to main content. The first two are links to anchors in the page. If I reload the page the extra content prior to the banner is no longer displayed and things look normal again. Joe Shiva Kumar H R wrote: I have been committing some patches in Admin Console, but most of them have been local to Plan Creator portlet. On the server that's built on my machine (which has all the commits I have done so far + trunk code as on 26-Jun), the portlets that you mention (Web App WARs, Deploy New, Plugins, Server Logs) seem to be working fine (I tested using IE 6 Firefox 3). I will update my trunk and do a clean build, and see if I face any problems. On Thu, Jul 10, 2008 at 9:59 PM, Jarek Gawor [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: Hi, I just built the latest code tried to use the admin console and the layout in a few places (e.g. Web App WARs, Deploy New, Plugins, Server Logs) is screwed up. I see similar layout issues on Firefox (2.0) and IE. It does not seem like people are checking how things look in the console before committing the patches... Jarek -- Thanks, Shiva -- Shrey Banga Bachelor of Technology, III year Department of Electrical Engineering Indian Institute of Technology Roorkee -- Shrey Banga Bachelor of Technology, III year Department of Electrical Engineering Indian Institute of Technology Roorkee
Re: recent console changes in trunk
I don't think that can be done because pluto.css is loaded after main.css and it'll override any previous css styles for label. On Tue, Jul 15, 2008 at 6:55 PM, Joe Bohn [EMAIL PROTECTED] wrote: YunFeng Ma wrote: I tested the console in FireFox 2.x and IE 6.x after removing label tag in pluto.css and didn't see any problem. I prefer to remove it before G v2.1.2 is out of the door. I agree we want this fixed before we ship 2.1.2. However, I'm not sure modifying pluto.css is the best way to go. All of our style sheet changes for the console have typically been restricted to main.css. I think pluto.css is pretty much a direct copy from pluto. It might be easier to maintain if we continue to keep our changes limited to main.css. Can you try fixing this by adding a new label entry in main.css without float:left? Joe -- Yun Feng Shrey Banga wrote: I've opened a JIRA (GERONIMO-4204) for this issue. I need to know if the float:left for labels in pluto.css is needed and where because it affects all Admin Console portlets. On Mon, Jul 14, 2008 at 3:09 PM, Shrey Banga [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: After rebuilding the server I too faced the same problem and on further inspection I could make out the problem was because the patch GERONIMO-4081 Accessibility issues (at revision: 675083) has surrounded text by label tags, which have a float:left property set in pluto.css which I suppose is causing the problems. I'm not sure why the float:left is needed in the first place but in case removing it isn't possible, we could create a separate css for the console and import it in the affected portlets. On Fri, Jul 11, 2008 at 6:50 PM, Joe Bohn [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: In addition to that quirk, there are other layout issues I'm seeing too (these are probably the ones Jarek was seeing). For example: - Server Log Viewer layout is all messed up - All of the Applications views (WARs, EARs, etc...) have the layout all messed up - Plugin portlets layout is messed up Those are just a few that I noticed clicking on the top level portlet pages. There may be other problems when you drill down into some of the pages to alternate jsps. Joe Joe Bohn wrote: I'm seeing strange things in trunk as well. Using firefox on a tomcat image I see the following prior to the banner in the console. I don't see this in safari: Skip to navigation Skip to main content The access keys for this page are: * ALT plus 1 skips to navigation. * ALT plus 2 skips to main content. The first two are links to anchors in the page. If I reload the page the extra content prior to the banner is no longer displayed and things look normal again. Joe Shiva Kumar H R wrote: I have been committing some patches in Admin Console, but most of them have been local to Plan Creator portlet. On the server that's built on my machine (which has all the commits I have done so far + trunk code as on 26-Jun), the portlets that you mention (Web App WARs, Deploy New, Plugins, Server Logs) seem to be working fine (I tested using IE 6 Firefox 3). I will update my trunk and do a clean build, and see if I face any problems. On Thu, Jul 10, 2008 at 9:59 PM, Jarek Gawor [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: Hi, I just built the latest code tried to use the admin console and the layout in a few places (e.g. Web App WARs, Deploy New, Plugins, Server Logs) is screwed up. I see similar layout issues on Firefox (2.0) and IE. It does not seem like people are checking how things look in the console before committing the patches... Jarek -- Thanks, Shiva -- Shrey Banga Bachelor of Technology, III year Department of Electrical Engineering Indian Institute of Technology Roorkee -- Shrey Banga Bachelor of Technology, III year Department of
Re: recent console changes in trunk
I searched the keyword label in all JSP files in an older revision which is before all the accessibility patches came in, and found no occurrences of label element. So I assume it's probably safe to remove the label style in pluto.css. Anyway it's safer if we can use main.css to override pluto.css. I can check this out later on. 2008/7/15 Joe Bohn [EMAIL PROTECTED]: YunFeng Ma wrote: I tested the console in FireFox 2.x and IE 6.x after removing label tag in pluto.css and didn't see any problem. I prefer to remove it before G v2.1.2 is out of the door. I agree we want this fixed before we ship 2.1.2. However, I'm not sure modifying pluto.css is the best way to go. All of our style sheet changes for the console have typically been restricted to main.css. I think pluto.css is pretty much a direct copy from pluto. It might be easier to maintain if we continue to keep our changes limited to main.css. Can you try fixing this by adding a new label entry in main.css without float:left? Joe
Re: recent console changes in trunk
Shrey Banga wrote: I don't think that can be done because pluto.css is loaded after main.css and it'll override any previous css styles for label. I'm not sure about what the correct processing is supposed to be for the style sheets ... but I made the change locally to add in a label entry in main.css without the float:left and it resolved the problem for me. I can go ahead and check this in if you'd like. In general I don't like duplication but I like have most if not all of the settings we care about in one place. Joe On Tue, Jul 15, 2008 at 6:55 PM, Joe Bohn [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: YunFeng Ma wrote: I tested the console in FireFox 2.x and IE 6.x after removing label tag in pluto.css and didn't see any problem. I prefer to remove it before G v2.1.2 is out of the door. I agree we want this fixed before we ship 2.1.2. http://2.1.2. However, I'm not sure modifying pluto.css is the best way to go. All of our style sheet changes for the console have typically been restricted to main.css. I think pluto.css is pretty much a direct copy from pluto. It might be easier to maintain if we continue to keep our changes limited to main.css. Can you try fixing this by adding a new label entry in main.css without float:left? Joe -- Yun Feng Shrey Banga wrote: I've opened a JIRA (GERONIMO-4204) for this issue. I need to know if the float:left for labels in pluto.css is needed and where because it affects all Admin Console portlets. On Mon, Jul 14, 2008 at 3:09 PM, Shrey Banga [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: After rebuilding the server I too faced the same problem and on further inspection I could make out the problem was because the patch GERONIMO-4081 Accessibility issues (at revision: 675083) has surrounded text by label tags, which have a float:left property set in pluto.css which I suppose is causing the problems. I'm not sure why the float:left is needed in the first place but in case removing it isn't possible, we could create a separate css for the console and import it in the affected portlets. On Fri, Jul 11, 2008 at 6:50 PM, Joe Bohn [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: In addition to that quirk, there are other layout issues I'm seeing too (these are probably the ones Jarek was seeing). For example: - Server Log Viewer layout is all messed up - All of the Applications views (WARs, EARs, etc...) have the layout all messed up - Plugin portlets layout is messed up Those are just a few that I noticed clicking on the top level portlet pages. There may be other problems when you drill down into some of the pages to alternate jsps. Joe Joe Bohn wrote: I'm seeing strange things in trunk as well. Using firefox on a tomcat image I see the following prior to the banner in the console. I don't see this in safari: Skip to navigation Skip to main content The access keys for this page are: * ALT plus 1 skips to navigation. * ALT plus 2 skips to main content. The first two are links to anchors in the page. If I reload the page the extra content prior to the banner is no longer displayed and things look normal again. Joe Shiva Kumar H R wrote: I have been committing some patches in Admin Console, but most of them have been local to Plan Creator portlet. On the server that's built on my machine (which has all the commits I have done so far + trunk code as on 26-Jun), the portlets that you mention (Web App WARs, Deploy New, Plugins, Server Logs) seem to be working
Re: recent console changes in trunk
Joe Bohn wrote: Shrey Banga wrote: I don't think that can be done because pluto.css is loaded after main.css and it'll override any previous css styles for label. I'm not sure about what the correct processing is supposed to be for the style sheets ... but I made the change locally to add in a label entry in main.css without the float:left and it resolved the problem for me. I can go ahead and check this in if you'd like. In general I don't like duplication but I like have most if not all of the settings we care about in one place. Oops ... I just realized that I made the changes in branches/2.1 (which I'm not even sure ever exhibited the problem). So my test may be invalid. Also, it may not be that hot of an issue for 2.1.2 ... for some reason I forgot that the issues were were seeing were in trunk. :-) I'm checking the change out on trunk now ... will let you know the results soon. Joe Joe On Tue, Jul 15, 2008 at 6:55 PM, Joe Bohn [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: YunFeng Ma wrote: I tested the console in FireFox 2.x and IE 6.x after removing label tag in pluto.css and didn't see any problem. I prefer to remove it before G v2.1.2 is out of the door. I agree we want this fixed before we ship 2.1.2. http://2.1.2. However, I'm not sure modifying pluto.css is the best way to go. All of our style sheet changes for the console have typically been restricted to main.css. I think pluto.css is pretty much a direct copy from pluto. It might be easier to maintain if we continue to keep our changes limited to main.css. Can you try fixing this by adding a new label entry in main.css without float:left? Joe -- Yun Feng Shrey Banga wrote: I've opened a JIRA (GERONIMO-4204) for this issue. I need to know if the float:left for labels in pluto.css is needed and where because it affects all Admin Console portlets. On Mon, Jul 14, 2008 at 3:09 PM, Shrey Banga [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: After rebuilding the server I too faced the same problem and on further inspection I could make out the problem was because the patch GERONIMO-4081 Accessibility issues (at revision: 675083) has surrounded text by label tags, which have a float:left property set in pluto.css which I suppose is causing the problems. I'm not sure why the float:left is needed in the first place but in case removing it isn't possible, we could create a separate css for the console and import it in the affected portlets. On Fri, Jul 11, 2008 at 6:50 PM, Joe Bohn [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: In addition to that quirk, there are other layout issues I'm seeing too (these are probably the ones Jarek was seeing). For example: - Server Log Viewer layout is all messed up - All of the Applications views (WARs, EARs, etc...) have the layout all messed up - Plugin portlets layout is messed up Those are just a few that I noticed clicking on the top level portlet pages. There may be other problems when you drill down into some of the pages to alternate jsps. Joe Joe Bohn wrote: I'm seeing strange things in trunk as well. Using firefox on a tomcat image I see the following prior to the banner in the console. I don't see this in safari: Skip to navigation Skip to main content The access keys for this page are: * ALT plus 1 skips to navigation. * ALT plus 2 skips to main content. The first two are links to anchors in the page. If I reload the page the extra content prior to the banner is no longer displayed and things look normal again. Joe Shiva Kumar H R wrote: I have been committing some patches in Admin Console, but most of them have
Re: recent console changes in trunk
Joe Bohn wrote: Joe Bohn wrote: Shrey Banga wrote: I don't think that can be done because pluto.css is loaded after main.css and it'll override any previous css styles for label. I'm not sure about what the correct processing is supposed to be for the style sheets ... but I made the change locally to add in a label entry in main.css without the float:left and it resolved the problem for me. I can go ahead and check this in if you'd like. In general I don't like duplication but I like have most if not all of the settings we care about in one place. Oops ... I just realized that I made the changes in branches/2.1 (which I'm not even sure ever exhibited the problem). So my test may be invalid. Also, it may not be that hot of an issue for 2.1.2 ... for some reason I forgot that the issues were were seeing were in trunk. :-) I'm checking the change out on trunk now ... will let you know the results soon. Joe Grrr I rebuilt too much in trunk, picked up the latest (just published) snapshots of openejb and now I can't even get the server to start (MEJB issues). I guess I won't be able to check out the change in main vs. pluto.css very soon. My level of concern is much lower anyway now that I am recalled this is not an issue for 2.1.2 (at least not yet). Joe Joe On Tue, Jul 15, 2008 at 6:55 PM, Joe Bohn [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: YunFeng Ma wrote: I tested the console in FireFox 2.x and IE 6.x after removing label tag in pluto.css and didn't see any problem. I prefer to remove it before G v2.1.2 is out of the door. I agree we want this fixed before we ship 2.1.2. http://2.1.2. However, I'm not sure modifying pluto.css is the best way to go. All of our style sheet changes for the console have typically been restricted to main.css. I think pluto.css is pretty much a direct copy from pluto. It might be easier to maintain if we continue to keep our changes limited to main.css. Can you try fixing this by adding a new label entry in main.css without float:left? Joe -- Yun Feng Shrey Banga wrote: I've opened a JIRA (GERONIMO-4204) for this issue. I need to know if the float:left for labels in pluto.css is needed and where because it affects all Admin Console portlets. On Mon, Jul 14, 2008 at 3:09 PM, Shrey Banga [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: After rebuilding the server I too faced the same problem and on further inspection I could make out the problem was because the patch GERONIMO-4081 Accessibility issues (at revision: 675083) has surrounded text by label tags, which have a float:left property set in pluto.css which I suppose is causing the problems. I'm not sure why the float:left is needed in the first place but in case removing it isn't possible, we could create a separate css for the console and import it in the affected portlets. On Fri, Jul 11, 2008 at 6:50 PM, Joe Bohn [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: In addition to that quirk, there are other layout issues I'm seeing too (these are probably the ones Jarek was seeing). For example: - Server Log Viewer layout is all messed up - All of the Applications views (WARs, EARs, etc...) have the layout all messed up - Plugin portlets layout is messed up Those are just a few that I noticed clicking on the top level portlet pages. There may be other problems when you drill down into some of the pages to alternate jsps. Joe Joe Bohn wrote: I'm seeing strange things in trunk as well. Using firefox on a tomcat image I see the following prior to the banner in the console. I don't see this in safari: Skip to navigation Skip to main content The access keys for this page are: * ALT plus 1 skips to navigation. * ALT plus 2 skips to main content. The first two are links to anchors in the page. If I reload
Re: recent console changes in trunk
I tried changing it locally and it doesn't work- gets overridden by pluto.css (firebug confirms that) If label wasn't used anywhere until now we can modify pluto.css. Why is it there anyway? On Tue, Jul 15, 2008 at 8:45 PM, Joe Bohn [EMAIL PROTECTED] wrote: Joe Bohn wrote: Joe Bohn wrote: Shrey Banga wrote: I don't think that can be done because pluto.css is loaded after main.css and it'll override any previous css styles for label. I'm not sure about what the correct processing is supposed to be for the style sheets ... but I made the change locally to add in a label entry in main.css without the float:left and it resolved the problem for me. I can go ahead and check this in if you'd like. In general I don't like duplication but I like have most if not all of the settings we care about in one place. Oops ... I just realized that I made the changes in branches/2.1 (which I'm not even sure ever exhibited the problem). So my test may be invalid. Also, it may not be that hot of an issue for 2.1.2 ... for some reason I forgot that the issues were were seeing were in trunk. :-) I'm checking the change out on trunk now ... will let you know the results soon. Joe Grrr I rebuilt too much in trunk, picked up the latest (just published) snapshots of openejb and now I can't even get the server to start (MEJB issues). I guess I won't be able to check out the change in main vs. pluto.css very soon. My level of concern is much lower anyway now that I am recalled this is not an issue for 2.1.2 (at least not yet). Joe Joe On Tue, Jul 15, 2008 at 6:55 PM, Joe Bohn [EMAIL PROTECTED]mailto: [EMAIL PROTECTED] wrote: YunFeng Ma wrote: I tested the console in FireFox 2.x and IE 6.x after removing label tag in pluto.css and didn't see any problem. I prefer to remove it before G v2.1.2 is out of the door. I agree we want this fixed before we ship 2.1.2. http://2.1.2. However, I'm not sure modifying pluto.css is the best way to go. All of our style sheet changes for the console have typically been restricted to main.css. I think pluto.css is pretty much a direct copy from pluto. It might be easier to maintain if we continue to keep our changes limited to main.css. Can you try fixing this by adding a new label entry in main.css without float:left? Joe -- Yun Feng Shrey Banga wrote: I've opened a JIRA (GERONIMO-4204) for this issue. I need to know if the float:left for labels in pluto.css is needed and where because it affects all Admin Console portlets. On Mon, Jul 14, 2008 at 3:09 PM, Shrey Banga [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: After rebuilding the server I too faced the same problem and on further inspection I could make out the problem was because the patch GERONIMO-4081 Accessibility issues (at revision: 675083) has surrounded text by label tags, which have a float:left property set in pluto.css which I suppose is causing the problems. I'm not sure why the float:left is needed in the first place but in case removing it isn't possible, we could create a separate css for the console and import it in the affected portlets. On Fri, Jul 11, 2008 at 6:50 PM, Joe Bohn [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: In addition to that quirk, there are other layout issues I'm seeing too (these are probably the ones Jarek was seeing). For example: - Server Log Viewer layout is all messed up - All of the Applications views (WARs, EARs, etc...) have the layout all messed up - Plugin portlets layout is messed up Those are just a few that I noticed clicking on the top level portlet pages. There may be other problems when you drill down into some of the pages to alternate jsps. Joe Joe Bohn wrote: I'm seeing strange things in trunk as well. Using firefox on a tomcat image I see the following prior to the banner in the console. I don't see this in safari: Skip to navigation Skip to main content The access keys for this page
Re: recent console changes in trunk
I just examined these two css carefully and found that the only overlapping style is body, and the body style in main.css is actuall void. So how about this - remove the body style in main.css, and then exhange their order in console\console-portal-driver\src\main\webapp\WEB-INF\themes\head.jsp? 2008/7/15 Shrey Banga [EMAIL PROTECTED]: I tried changing it locally and it doesn't work- gets overridden by pluto.css (firebug confirms that) If label wasn't used anywhere until now we can modify pluto.css. Why is it there anyway? On Tue, Jul 15, 2008 at 8:45 PM, Joe Bohn [EMAIL PROTECTED] wrote: Joe Bohn wrote: Joe Bohn wrote: Shrey Banga wrote: I don't think that can be done because pluto.css is loaded after main.css and it'll override any previous css styles for label. I'm not sure about what the correct processing is supposed to be for the style sheets ... but I made the change locally to add in a label entry in main.css without the float:left and it resolved the problem for me. I can go ahead and check this in if you'd like. In general I don't like duplication but I like have most if not all of the settings we care about in one place. Oops ... I just realized that I made the changes in branches/2.1 (which I'm not even sure ever exhibited the problem). So my test may be invalid. Also, it may not be that hot of an issue for 2.1.2 ... for some reason I forgot that the issues were were seeing were in trunk. :-) I'm checking the change out on trunk now ... will let you know the results soon. Joe Grrr I rebuilt too much in trunk, picked up the latest (just published) snapshots of openejb and now I can't even get the server to start (MEJB issues). I guess I won't be able to check out the change in main vs. pluto.css very soon. My level of concern is much lower anyway now that I am recalled this is not an issue for 2.1.2 (at least not yet). Joe Joe On Tue, Jul 15, 2008 at 6:55 PM, Joe Bohn [EMAIL PROTECTED]mailto: [EMAIL PROTECTED] wrote: YunFeng Ma wrote: I tested the console in FireFox 2.x and IE 6.x after removing label tag in pluto.css and didn't see any problem. I prefer to remove it before G v2.1.2 is out of the door. I agree we want this fixed before we ship 2.1.2. http://2.1.2. However, I'm not sure modifying pluto.css is the best way to go. All of our style sheet changes for the console have typically been restricted to main.css. I think pluto.css is pretty much a direct copy from pluto. It might be easier to maintain if we continue to keep our changes limited to main.css. Can you try fixing this by adding a new label entry in main.css without float:left? Joe -- Yun Feng Shrey Banga wrote: I've opened a JIRA (GERONIMO-4204) for this issue. I need to know if the float:left for labels in pluto.css is needed and where because it affects all Admin Console portlets. On Mon, Jul 14, 2008 at 3:09 PM, Shrey Banga [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: After rebuilding the server I too faced the same problem and on further inspection I could make out the problem was because the patch GERONIMO-4081 Accessibility issues (at revision: 675083) has surrounded text by label tags, which have a float:left property set in pluto.css which I suppose is causing the problems. I'm not sure why the float:left is needed in the first place but in case removing it isn't possible, we could create a separate css for the console and import it in the affected portlets. On Fri, Jul 11, 2008 at 6:50 PM, Joe Bohn [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: In addition to that quirk, there are other layout issues I'm seeing too (these are probably the ones Jarek was seeing). For example: - Server Log Viewer layout is all messed up - All of the Applications views (WARs, EARs, etc...) have the layout all messed up - Plugin portlets layout is messed up Those are just a few that I noticed clicking on the top level portlet pages. There may be other problems when you drill down into some of the pages to alternate jsps. Joe Joe Bohn wrote: I'm seeing strange things in trunk as
Re: recent console changes in trunk
Sorry I overlooked the fact that there is another overlapping style td. So my last suggestion is not ideal... How about just to add a label style in head.jsp to override the one in pluto.css? It's dirty though... 2008/7/15 Jack [EMAIL PROTECTED]: I just examined these two css carefully and found that the only overlapping style is body, and the body style in main.css is actuall void. So how about this - remove the body style in main.css, and then exhange their order in console\console-portal-driver\src\main\webapp\WEB-INF\themes\head.jsp? 2008/7/15 Shrey Banga [EMAIL PROTECTED]: I tried changing it locally and it doesn't work- gets overridden by pluto.css (firebug confirms that) If label wasn't used anywhere until now we can modify pluto.css. Why is it there anyway? On Tue, Jul 15, 2008 at 8:45 PM, Joe Bohn [EMAIL PROTECTED] wrote: Joe Bohn wrote: Joe Bohn wrote: Shrey Banga wrote: I don't think that can be done because pluto.css is loaded after main.css and it'll override any previous css styles for label. I'm not sure about what the correct processing is supposed to be for the style sheets ... but I made the change locally to add in a label entry in main.css without the float:left and it resolved the problem for me. I can go ahead and check this in if you'd like. In general I don't like duplication but I like have most if not all of the settings we care about in one place. Oops ... I just realized that I made the changes in branches/2.1 (which I'm not even sure ever exhibited the problem). So my test may be invalid. Also, it may not be that hot of an issue for 2.1.2 ... for some reason I forgot that the issues were were seeing were in trunk. :-) I'm checking the change out on trunk now ... will let you know the results soon. Joe Grrr I rebuilt too much in trunk, picked up the latest (just published) snapshots of openejb and now I can't even get the server to start (MEJB issues). I guess I won't be able to check out the change in main vs. pluto.css very soon. My level of concern is much lower anyway now that I am recalled this is not an issue for 2.1.2 (at least not yet). Joe Joe On Tue, Jul 15, 2008 at 6:55 PM, Joe Bohn [EMAIL PROTECTED]mailto: [EMAIL PROTECTED] wrote: YunFeng Ma wrote: I tested the console in FireFox 2.x and IE 6.x after removing label tag in pluto.css and didn't see any problem. I prefer to remove it before G v2.1.2 is out of the door. I agree we want this fixed before we ship 2.1.2. http://2.1.2. However, I'm not sure modifying pluto.css is the best way to go. All of our style sheet changes for the console have typically been restricted to main.css. I think pluto.css is pretty much a direct copy from pluto. It might be easier to maintain if we continue to keep our changes limited to main.css. Can you try fixing this by adding a new label entry in main.css without float:left? Joe -- Yun Feng Shrey Banga wrote: I've opened a JIRA (GERONIMO-4204) for this issue. I need to know if the float:left for labels in pluto.css is needed and where because it affects all Admin Console portlets. On Mon, Jul 14, 2008 at 3:09 PM, Shrey Banga [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: After rebuilding the server I too faced the same problem and on further inspection I could make out the problem was because the patch GERONIMO-4081 Accessibility issues (at revision: 675083) has surrounded text by label tags, which have a float:left property set in pluto.css which I suppose is causing the problems. I'm not sure why the float:left is needed in the first place but in case removing it isn't possible, we could create a separate css for the console and import it in the affected portlets. On Fri, Jul 11, 2008 at 6:50 PM, Joe Bohn [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: In addition to that quirk, there are other layout issues I'm seeing too (these are probably the ones Jarek was seeing). For example: - Server Log Viewer layout is all messed up - All of the Applications views (WARs, EARs, etc...) have the layout all messed up - Plugin portlets layout is messed up Those are just a few that I noticed clicking on the top level portlet pages. There may be other problems when
Re: recent console changes in trunk
Shrey Banga wrote: I tried changing it locally and it doesn't work- gets overridden by pluto.css (firebug confirms that) If label wasn't used anywhere until now we can modify pluto.css. Why is it there anyway? I have no idea why pluto.css is there. Perhaps we should just merge the missing elements from pluto.css into main.css and see if we can remove pluto.css entirely. I wonder if there are any cases where we use pluto.css without main.css. Joe On Tue, Jul 15, 2008 at 8:45 PM, Joe Bohn [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: Joe Bohn wrote: Joe Bohn wrote: Shrey Banga wrote: I don't think that can be done because pluto.css is loaded after main.css and it'll override any previous css styles for label. I'm not sure about what the correct processing is supposed to be for the style sheets ... but I made the change locally to add in a label entry in main.css without the float:left and it resolved the problem for me. I can go ahead and check this in if you'd like. In general I don't like duplication but I like have most if not all of the settings we care about in one place. Oops ... I just realized that I made the changes in branches/2.1 (which I'm not even sure ever exhibited the problem). So my test may be invalid. Also, it may not be that hot of an issue for 2.1.2 ... for some reason I forgot that the issues were were seeing were in trunk. :-) I'm checking the change out on trunk now ... will let you know the results soon. Joe Grrr I rebuilt too much in trunk, picked up the latest (just published) snapshots of openejb and now I can't even get the server to start (MEJB issues). I guess I won't be able to check out the change in main vs. pluto.css very soon. My level of concern is much lower anyway now that I am recalled this is not an issue for 2.1.2 (at least not yet). Joe Joe On Tue, Jul 15, 2008 at 6:55 PM, Joe Bohn [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: YunFeng Ma wrote: I tested the console in FireFox 2.x and IE 6.x after removing label tag in pluto.css and didn't see any problem. I prefer to remove it before G v2.1.2 is out of the door. I agree we want this fixed before we ship 2.1.2. http://2.1.2. http://2.1.2. However, I'm not sure modifying pluto.css is the best way to go. All of our style sheet changes for the console have typically been restricted to main.css. I think pluto.css is pretty much a direct copy from pluto. It might be easier to maintain if we continue to keep our changes limited to main.css. Can you try fixing this by adding a new label entry in main.css without float:left? Joe -- Yun Feng Shrey Banga wrote: I've opened a JIRA (GERONIMO-4204) for this issue. I need to know if the float:left for labels in pluto.css is needed and where because it affects all Admin Console portlets. On Mon, Jul 14, 2008 at 3:09 PM, Shrey Banga [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: After rebuilding the server I too faced the same problem and on further inspection I could make out the problem was because the patch GERONIMO-4081 Accessibility issues (at revision: 675083) has surrounded text by label tags, which have a float:left property set in pluto.css which I suppose is causing the problems. I'm not sure why the float:left is needed in the first place but
Re: recent console changes in trunk
After rebuilding the server I too faced the same problem and on further inspection I could make out the problem was because the patch GERONIMO-4081 Accessibility issues (at revision: 675083) has surrounded text by label tags, which have a float:left property set in pluto.css which I suppose is causing the problems. I'm not sure why the float:left is needed in the first place but in case removing it isn't possible, we could create a separate css for the console and import it in the affected portlets. On Fri, Jul 11, 2008 at 6:50 PM, Joe Bohn [EMAIL PROTECTED] wrote: In addition to that quirk, there are other layout issues I'm seeing too (these are probably the ones Jarek was seeing). For example: - Server Log Viewer layout is all messed up - All of the Applications views (WARs, EARs, etc...) have the layout all messed up - Plugin portlets layout is messed up Those are just a few that I noticed clicking on the top level portlet pages. There may be other problems when you drill down into some of the pages to alternate jsps. Joe Joe Bohn wrote: I'm seeing strange things in trunk as well. Using firefox on a tomcat image I see the following prior to the banner in the console. I don't see this in safari: Skip to navigation Skip to main content The access keys for this page are: * ALT plus 1 skips to navigation. * ALT plus 2 skips to main content. The first two are links to anchors in the page. If I reload the page the extra content prior to the banner is no longer displayed and things look normal again. Joe Shiva Kumar H R wrote: I have been committing some patches in Admin Console, but most of them have been local to Plan Creator portlet. On the server that's built on my machine (which has all the commits I have done so far + trunk code as on 26-Jun), the portlets that you mention (Web App WARs, Deploy New, Plugins, Server Logs) seem to be working fine (I tested using IE 6 Firefox 3). I will update my trunk and do a clean build, and see if I face any problems. On Thu, Jul 10, 2008 at 9:59 PM, Jarek Gawor [EMAIL PROTECTED] mailto: [EMAIL PROTECTED] wrote: Hi, I just built the latest code tried to use the admin console and the layout in a few places (e.g. Web App WARs, Deploy New, Plugins, Server Logs) is screwed up. I see similar layout issues on Firefox (2.0) and IE. It does not seem like people are checking how things look in the console before committing the patches... Jarek -- Thanks, Shiva -- Shrey Banga Bachelor of Technology, III year Department of Electrical Engineering Indian Institute of Technology Roorkee
Re: recent console changes in trunk
I have been committing some patches in Admin Console, but most of them have been local to Plan Creator portlet. On the server that's built on my machine (which has all the commits I have done so far + trunk code as on 26-Jun), the portlets that you mention (Web App WARs, Deploy New, Plugins, Server Logs) seem to be working fine (I tested using IE 6 Firefox 3). I will update my trunk and do a clean build, and see if I face any problems. On Thu, Jul 10, 2008 at 9:59 PM, Jarek Gawor [EMAIL PROTECTED] wrote: Hi, I just built the latest code tried to use the admin console and the layout in a few places (e.g. Web App WARs, Deploy New, Plugins, Server Logs) is screwed up. I see similar layout issues on Firefox (2.0) and IE. It does not seem like people are checking how things look in the console before committing the patches... Jarek -- Thanks, Shiva
Re: recent console changes in trunk
I'm seeing strange things in trunk as well. Using firefox on a tomcat image I see the following prior to the banner in the console. I don't see this in safari: Skip to navigation Skip to main content The access keys for this page are: * ALT plus 1 skips to navigation. * ALT plus 2 skips to main content. The first two are links to anchors in the page. If I reload the page the extra content prior to the banner is no longer displayed and things look normal again. Joe Shiva Kumar H R wrote: I have been committing some patches in Admin Console, but most of them have been local to Plan Creator portlet. On the server that's built on my machine (which has all the commits I have done so far + trunk code as on 26-Jun), the portlets that you mention (Web App WARs, Deploy New, Plugins, Server Logs) seem to be working fine (I tested using IE 6 Firefox 3). I will update my trunk and do a clean build, and see if I face any problems. On Thu, Jul 10, 2008 at 9:59 PM, Jarek Gawor [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: Hi, I just built the latest code tried to use the admin console and the layout in a few places (e.g. Web App WARs, Deploy New, Plugins, Server Logs) is screwed up. I see similar layout issues on Firefox (2.0) and IE. It does not seem like people are checking how things look in the console before committing the patches... Jarek -- Thanks, Shiva
Re: recent console changes in trunk
In addition to that quirk, there are other layout issues I'm seeing too (these are probably the ones Jarek was seeing). For example: - Server Log Viewer layout is all messed up - All of the Applications views (WARs, EARs, etc...) have the layout all messed up - Plugin portlets layout is messed up Those are just a few that I noticed clicking on the top level portlet pages. There may be other problems when you drill down into some of the pages to alternate jsps. Joe Joe Bohn wrote: I'm seeing strange things in trunk as well. Using firefox on a tomcat image I see the following prior to the banner in the console. I don't see this in safari: Skip to navigation Skip to main content The access keys for this page are: * ALT plus 1 skips to navigation. * ALT plus 2 skips to main content. The first two are links to anchors in the page. If I reload the page the extra content prior to the banner is no longer displayed and things look normal again. Joe Shiva Kumar H R wrote: I have been committing some patches in Admin Console, but most of them have been local to Plan Creator portlet. On the server that's built on my machine (which has all the commits I have done so far + trunk code as on 26-Jun), the portlets that you mention (Web App WARs, Deploy New, Plugins, Server Logs) seem to be working fine (I tested using IE 6 Firefox 3). I will update my trunk and do a clean build, and see if I face any problems. On Thu, Jul 10, 2008 at 9:59 PM, Jarek Gawor [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: Hi, I just built the latest code tried to use the admin console and the layout in a few places (e.g. Web App WARs, Deploy New, Plugins, Server Logs) is screwed up. I see similar layout issues on Firefox (2.0) and IE. It does not seem like people are checking how things look in the console before committing the patches... Jarek -- Thanks, Shiva