Bug 812416 - The skin from examples looks bad, too much whitespace
Summary: The skin from examples looks bad, too much whitespace
Keywords:
Status: VERIFIED
Alias: None
Product: JBoss Enterprise Portal Platform 5
Classification: JBoss
Component: Portal
Version: 5.2.1.CR02
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: 5.2.2.ER03
Assignee: Peter Palaga
QA Contact:
URL:
Whiteboard:
: 840457 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-04-13 16:32 UTC by Viliam Rockai
Modified: 2018-02-06 19:19 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed:
Type: Bug
Embargoed:


Attachments (Terms of Use)
skin screenshot (81.25 KB, image/jpeg)
2012-04-13 18:04 UTC, Viliam Rockai
no flags Details

Description Viliam Rockai 2012-04-13 16:32:49 UTC
see the attached screenshot.

Comment 1 Viliam Rockai 2012-04-13 18:04:39 UTC
Created attachment 577397 [details]
skin screenshot

Comment 2 Peter Palaga 2012-06-18 14:06:14 UTC
Please specify the steps to reproduce the described behavior.

Comment 3 Peter Palaga 2012-07-30 08:05:03 UTC
*** Bug 840457 has been marked as a duplicate of this bug. ***

Comment 4 Peter Palaga 2012-07-30 08:16:57 UTC
The duplicate Bug 840457 by Viliam Rockai has added some pieces of information which are necessary to reproduce this bug:

> Description of problem:
> Simple skin looks bad on sample-portal (both have to be deployed
> from examples). Please see the attached screenshot. The problem
> consists of huge white space and corner decorations at the bottom
> of the page.
>
> Fix should be really easy, in 
>
> div.UIPageBody > div.UIPage 
> 
> change 
> 
> padding: 0 8px;
> 
> to
> 
> padding: 0 0;

Comment 5 Peter Palaga 2012-07-30 11:56:45 UTC
Fixed as proposed by Viliam. The padding: 0 8px rule was removed from /exo.portal.parent/examples/skins/simpleskin/src/main/webapp/skin/SimpleSkin/portal/webui/component/view/UIPage/Stylesheet.css.

Clearly the changed padding affects also /portal/classic. The change is acceptable, IMO.

Comment 6 Michal Vanco 2012-08-16 16:52:33 UTC
This was fixed and commited at ER03 release, changing target release and setting as verified.

Comment 7 mgottval 2012-08-27 16:52:52 UTC
Verified again in 5.2.2.CR01.


Note You need to log in before you can comment on or make changes to this bug.