Skip to content

Commit 62a916c

Browse files
committed
Plugins: Fix plugin modal scrollbar visibility.
Set height of scrollable container to exclude the modal footer so that the bottom of the scrollbar remains visible and accessible. Simplify styling by eliminating small viewport height adjustment. Change modal positioning in mobile viewports to ensure entire modal is visible with a masked margin. Developed in WordPress#11957 Props projektisle, alanjacobmathew, joedolson, sabernhardt. Fixes #65334. git-svn-id: https://develop.svn.wordpress.org/trunk@62691 602fd350-edb4-49c9-b593-d223f7449a82
1 parent a71fb9b commit 62a916c

2 files changed

Lines changed: 2 additions & 7 deletions

File tree

src/js/_enqueues/admin/plugin-install.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ jQuery( function( $ ) {
2020

2121
window.tb_position = function() {
2222
var width = $( window ).width(),
23-
H = $( window ).height() - ( ( 792 < width ) ? 60 : 20 ),
23+
H = $( window ).height() - ( ( 792 < width ) ? 60 : 40 ),
2424
W = ( 792 < width ) ? 772 : width - 20;
2525

2626
tbWindow = $( '#TB_window' );

src/wp-admin/css/common.css

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2804,7 +2804,7 @@ div.action-links {
28042804
#plugin-information-scrollable {
28052805
overflow: auto;
28062806
-webkit-overflow-scrolling: touch;
2807-
height: 100%;
2807+
height: calc(100% - 67px);
28082808
}
28092809

28102810
#plugin-information-title {
@@ -3204,11 +3204,6 @@ div.action-links {
32043204
#plugin-information .fyi small {
32053205
display: none;
32063206
}
3207-
3208-
#plugin-information-footer {
3209-
padding: 12px 16px 0;
3210-
height: 46px;
3211-
}
32123207
}
32133208

32143209
/* Thickbox for the Plugin details modal. */

0 commit comments

Comments
 (0)