Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.
Bug 1446563 - [trello mnJMUAey] Better to left align the added saas offering
[trello mnJMUAey] Better to left align the added saas offering
Status: CLOSED CURRENTRELEASE
Product: OpenShift Container Platform
Classification: Red Hat
Component: Management Console (Show other bugs)
3.6.0
Unspecified Unspecified
medium Severity low
: ---
: ---
Assigned To: Jeff Phillips
XiaochuanWang
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2017-04-28 06:46 EDT by Xingxing Xia
Modified: 2017-08-16 15 EDT (History)
9 users (show)

See Also:
Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2017-07-25 09:56:21 EDT
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
xxia: needinfo-


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


External Trackers
Tracker ID Priority Status Summary Last Updated
Red Hat Product Errata RHEA-2017:1716 normal SHIPPED_LIVE Red Hat OpenShift Container Platform 3.6 RPM Release Advisory 2017-08-10 05:02:50 EDT

  None (edit)
Description Xingxing Xia 2017-04-28 06:46:37 EDT
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 06:50 EDT
Created attachment 1274870 [details]
Added saas offering is center aligned
Comment 4 David Taylor 2017-04-28 10:35:15 EDT
Hi, this is a design decision.  Reassigning to a ux designer.
Comment 5 Samuel Padgett 2017-05-09 09:38:11 EDT
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 14:25:59 EDT
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 10:03:49 EDT
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 10:10:34 EDT
Please wait till the PR is merged, and then move to MODIFIED not ON_QA
Comment 9 Jeff Phillips 2017-05-25 07:06:33 EDT
PR has been merged
Comment 11 Xingxing Xia 2017-06-06 04:12:05 EDT
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 04:36 EDT
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 09:19:56 EDT
> 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 09:20 EDT
Created attachment 1285382 [details]
See more fix for mobile
Comment 15 Xingxing Xia 2017-06-06 23:10:45 EDT
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 09:07:32 EDT
(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.