Bug 1276216 - Pod terminal page didn't adjust automatically to fit its size with browser window
Summary: Pod terminal page didn't adjust automatically to fit its size with browser wi...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: OKD
Classification: Red Hat
Component: Management Console
Version: 3.x
Hardware: Unspecified
OS: Unspecified
high
low
Target Milestone: ---
: ---
Assignee: Steve Goodwin
QA Contact: Yadan Pei
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-10-29 06:25 UTC by Yadan Pei
Modified: 2019-12-10 01:27 UTC (History)
8 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-11-10 21:34:14 UTC
Target Upstream Version:
Embargoed:
stefw: needinfo-


Attachments (Terms of Use)
Button in lower left corner (34.98 KB, image/png)
2015-10-29 06:25 UTC, Yadan Pei
no flags Details
iPhone6Terminal (43.78 KB, image/png)
2016-11-10 06:49 UTC, Yadan Pei
no flags Details
Position of reconnect icon corrected (78.64 KB, image/png)
2017-06-21 15:13 UTC, Robb Hamilton
no flags Details

Description Yadan Pei 2015-10-29 06:25:05 UTC
Description of problem:
When browser window width is less than 702, the re-connect button on 'Terminal' page of pod was pushed to the lower left corner. 

Version-Release number of selected component (if applicable):
oc v1.0.6-1053-gbd6a762
kubernetes v1.2.0-alpha.1-1107-g4c8e6f4

How reproducible:
Always

Steps to Reproduce:
1.Create a template
$ oc create -f https://raw.githubusercontent.com/openshift/origin/master/examples/sample-app/application-template-stibuild.json
2. Create app from template from web console
3. Wait for all pods to be running
4. Open 'Terminal' page for one pod
5. Resize the browser window 

Actual results:
When browser window width is less than 702, the re-connect button on 'Terminal' page of pod was pushed to the lower left corner, see attachment

Expected results:
re-connect button should always shows at the right position 

Additional info:
702 is not a precise value, but it's very close to the correct one

Comment 1 Yadan Pei 2015-10-29 06:25:45 UTC
Created attachment 1087384 [details]
Button in lower left corner

Comment 2 Jessica Forrester 2016-02-01 20:32:27 UTC
Because the terminal-actions and terminal-wrapper divs are both inline-block, when you get to a certain screen resolution the second div (actions) drops to the next line, and then the relative positioning pushes it to the left, cutting it off.  I would propose moving the terminal-actions to the bottom left instead of the top right, we can then use position: absolute instead of position relative and it will always stay in the same position.

But since this is a change to a common widget I'd like sign-off from @stefw before moving forward with the fix.

Comment 3 Jessica Forrester 2016-11-08 16:19:11 UTC
This was fixed back in 3.3

Comment 4 Yadan Pei 2016-11-10 06:48:38 UTC
Checked on v1.4.0-alpha.1+43fbf69-133 & OCP 3.3.1.4

Pod Terminal are not fully shown on iPhone 6, please check attachment

Assigning back

Comment 5 Yadan Pei 2016-11-10 06:49:07 UTC
Created attachment 1219191 [details]
iPhone6Terminal

Comment 6 Yadan Pei 2016-11-10 06:51:46 UTC
//div[@class="terminal"] are not fully displayed when browser width changes to smaller

Comment 7 Jessica Forrester 2016-11-10 13:20:34 UTC
The terminal extending out is intentional, we require a minimum number of columns (80), you should be able to scroll over using the swipe gesture to see the rest of the terminal.

The location of the reconnect icon at this screen width is a little weird though, we'll take another look at that.

Comment 8 Jessica Forrester 2016-12-08 21:49:06 UTC
Steve can you take a look at the position of the reconnect icon at mobile.

Comment 9 Robb Hamilton 2017-06-21 15:13:01 UTC
Created attachment 1290191 [details]
Position of reconnect icon corrected

Was addressed in https://github.com/openshift/origin-web-console/commit/99e338cf2616fe992adfc9c02d123d01eee15c7a

Comment 10 Jessica Forrester 2017-06-21 17:17:03 UTC
origin bug, so moving to ON_QA

Comment 11 shahan 2017-06-22 07:11:57 UTC
Checked this issue in openshift v3.6.122, it was fixed, change this bug status to verified.


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