Bug 1446563 - [trello mnJMUAey] Better to left align the added saas offering
Summary: [trello mnJMUAey] Better to left align the added saas offering
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: Management Console
Version: 3.6.0
Hardware: Unspecified
OS: Unspecified
medium
low
Target Milestone: ---
: ---
Assignee: Jeff Phillips
QA Contact: XiaochuanWang
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-04-28 10:46 UTC by Xingxing Xia
Modified: 2017-08-16 19:51 UTC (History)
9 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-07-25 13:56:21 UTC
Target Upstream Version:
Embargoed:
xxia: needinfo-


Attachments (Terms of Use)
Added saas offering is center aligned (552.49 KB, image/png)
2017-04-28 10:50 UTC, Xingxing Xia
no flags Details
mobile-phone (236.73 KB, image/png)
2017-06-06 08:36 UTC, Xingxing Xia
no flags Details
See more fix for mobile (1.01 MB, image/png)
2017-06-06 13:20 UTC, Robb Hamilton
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHEA-2017:1716 0 normal SHIPPED_LIVE Red Hat OpenShift Container Platform 3.6 RPM Release Advisory 2017-08-10 09:02:50 UTC

Description Xingxing Xia 2017-04-28 10:46:37 UTC
Description of problem:
The added saas offering is center aligned. Better to left align, which looks better in display. See attachment

Version-Release number of selected component (if applicable):
openshift v3.6.50

How reproducible:
Always

Steps to Reproduce:
1. After Service Catalog is deployed, customize the saas offering section by:
$ cat saas-offering.js
window.OPENSHIFT_CONSTANTS.SAAS_OFFERINGS.push( {id: 5, title: 'XXIA Example App', icon: 'fa fa-cubes', url: 'https://www.redhat.com/en/technologies/management', description: 'this is added by xxia'})

$ vi master-config.yaml
...
assetConfig:
  extensionScripts:
  - saas-offering.js
...

Then restart master service

2. Go to home page, check the added saas offering "XXIA Example App"

Actual results:
2. It is center aligned

Expected results:
2. Better to left align

Additional info:

Comment 1 Xingxing Xia 2017-04-28 10:50:32 UTC
Created attachment 1274870 [details]
Added saas offering is center aligned

Comment 4 David Taylor 2017-04-28 14:35:15 UTC
Hi, this is a design decision.  Reassigning to a ux designer.

Comment 5 Samuel Padgett 2017-05-09 13:38:11 UTC
To me the issue is that the offerings take up too much space when they wrap to the second row. I suggest we deal with that as part of https://trello.com/c/h1BT3lyQ

Comment 6 Colleen Hart 2017-05-12 18:25:59 UTC
The design was altered and the Trello card was reassigned to a dev for implementation. Assigning this to Jeff as he will be implementing the new design.

Comment 7 Jeff Phillips 2017-05-24 14:03:49 UTC
The sass list has been updated to use a Show More / Show Less when there is more than one row. The console will be updated with the latest release of origin-web-catalog, PR https://github.com/openshift/origin-web-console/pull/1578

Comment 8 Jessica Forrester 2017-05-24 14:10:34 UTC
Please wait till the PR is merged, and then move to MODIFIED not ON_QA

Comment 9 Jeff Phillips 2017-05-25 11:06:33 UTC
PR has been merged

Comment 11 Xingxing Xia 2017-06-06 08:12:05 UTC
Checked in openshift v3.6.95
Now when more than 4 offerings, there is a "Show More" link. Clicking "Show More", the more offerings are left aligned
In mobile ipad, the result is same (one difference is one row has no more than 2 offerings as expected)
From perspective of left alignment, moving to Verified

Comment 12 Xingxing Xia 2017-06-06 08:36:08 UTC
Created attachment 1285288 [details]
mobile-phone

Per https://github.com/openshift/openshift-origin-design/pull/34/files#diff-088478d9c15e3488a412d1db527d6c72R38 , it says:
The same treatment should be used on mobile when more than one row of SaaS offerings are available (greater than 2).
Once more than two offerings exist, the down arrow with appear at the bottom allowing users to expand the section

I checked on mobile pad, it has "Show More" when more than 2 offerings.
But on mobile phone (such iPhone 6 Plus), no "Show More", all are shown vertically, see attachment "mobile-phone".
Should mobile phone have "Show More" or show all? Thanks

Comment 13 Robb Hamilton 2017-06-06 13:19:56 UTC
> I checked on mobile pad, it has "Show More" when more than 2 offerings.
But on mobile phone (such iPhone 6 Plus), no "Show More", all are shown vertically, see attachment "mobile-phone".
Should mobile phone have "Show More" or show all? Thanks

This has been fixed in https://github.com/openshift/origin-web-catalog/pull/261, but note that branch is still a work in progress.

Comment 14 Robb Hamilton 2017-06-06 13:20:29 UTC
Created attachment 1285382 [details]
See more fix for mobile

Comment 15 Xingxing Xia 2017-06-07 03:10:45 UTC
Thank you. Filed bug 1459391 just for issue track.

(In reply to Robb Hamilton from comment #14)
> Created attachment 1285382 [details]
> See more fix for mobile

Your attachment is in mobile pad, already said in comment 12, i.e. "Show More" in mobile pad was already good

Comment 16 Robb Hamilton 2017-06-07 13:07:32 UTC
(In reply to Xingxing Xia from comment #15)
> Thank you. Filed bug 1459391 just for issue track.
> 
> (In reply to Robb Hamilton from comment #14)
> > Created attachment 1285382 [details]
> > See more fix for mobile
> 
> Your attachment is in mobile pad, already said in comment 12, i.e. "Show
> More" in mobile pad was already good

Oops.  Sorry, Xingxing.  I will attach the proper screenshots in https://bugzilla.redhat.com/show_bug.cgi?id=1459391.


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