[go: up one dir, main page]

Page MenuHomePhabricator

Add padding to the popup [S]
Closed, ResolvedPublic

Description

The popup widget currently has no padding:

Screenshot.png (362×336 px, 26 KB)

The PopupButtonWidget is created in popup/init.js:

button = new OO.ui.PopupButtonWidget( {
	icon: 'info',
	framed: false,
	classes: [ 'ext-ipinfo-button' ]
} );

We can add padding by adding:

button = new OO.ui.PopupButtonWidget( {
	...,
	popup: {
		padded: true
	}
} );

...as described in the PopupButtonWidget documentation.

Event Timeline

Change 751969 had a related patch set uploaded (by Tchanders; author: Tchanders):

[mediawiki/extensions/IPInfo@master] Add padding to the popup

https://gerrit.wikimedia.org/r/751969

Tchanders renamed this task from Add padding to the popup to Add padding to the popup [S].Jan 6 2022, 5:02 PM

Change 751969 merged by jenkins-bot:

[mediawiki/extensions/IPInfo@master] Add padding to the popup

https://gerrit.wikimedia.org/r/751969

Padding now shows via testing on my local environment: See screen shot below:
I edited the skin on local box and padding was still in place. Tested on desktop Safari, Firefox, and Chrome.

Screen Shot 2022-01-12 at 6.37.54 PM.png (1×2 px, 894 KB)