jenkins-bot has submitted this change and it was merged. Change subject: Use new TabSelectWidget in link inspector ......................................................................
Use new TabSelectWidget in link inspector Don't use the full IndexLayout for now as it doesn't work easily in variable height situations. Depends on this feature being preset in MW core. Bug: T97878 Change-Id: I5d949bf69b0721b288c37c1f12210b0265658eb5 --- M modules/ve-mw/ui/inspectors/ve.ui.MWLinkAnnotationInspector.js M modules/ve-mw/ui/styles/inspectors/ve.ui.MWLinkAnnotationInspector.css 2 files changed, 5 insertions(+), 16 deletions(-) Approvals: Trevor Parscal: Looks good to me, approved jenkins-bot: Verified diff --git a/modules/ve-mw/ui/inspectors/ve.ui.MWLinkAnnotationInspector.js b/modules/ve-mw/ui/inspectors/ve.ui.MWLinkAnnotationInspector.js index 3172d40..1a86b06 100644 --- a/modules/ve-mw/ui/inspectors/ve.ui.MWLinkAnnotationInspector.js +++ b/modules/ve-mw/ui/inspectors/ve.ui.MWLinkAnnotationInspector.js @@ -48,11 +48,11 @@ } ); this.externalAnnotationInput = new ve.ui.MWExternalLinkAnnotationWidget(); - this.linkTypeSelect = new OO.ui.ButtonSelectWidget( { + this.linkTypeSelect = new OO.ui.TabSelectWidget( { classes: [ 've-ui-mwLinkAnnotationInspector-linkTypeSelect' ], items: [ - new OO.ui.ButtonOptionWidget( { framed: false, data: 'internal', label: ve.msg( 'visualeditor-linkinspector-button-link-internal' ) } ), - new OO.ui.ButtonOptionWidget( { framed: false, data: 'external', label: ve.msg( 'visualeditor-linkinspector-button-link-external' ) } ) + new OO.ui.TabOptionWidget( { data: 'internal', label: ve.msg( 'visualeditor-linkinspector-button-link-internal' ) } ), + new OO.ui.TabOptionWidget( { data: 'external', label: ve.msg( 'visualeditor-linkinspector-button-link-external' ) } ) ] } ); diff --git a/modules/ve-mw/ui/styles/inspectors/ve.ui.MWLinkAnnotationInspector.css b/modules/ve-mw/ui/styles/inspectors/ve.ui.MWLinkAnnotationInspector.css index 08f7d9e..85c2b18 100644 --- a/modules/ve-mw/ui/styles/inspectors/ve.ui.MWLinkAnnotationInspector.css +++ b/modules/ve-mw/ui/styles/inspectors/ve.ui.MWLinkAnnotationInspector.css @@ -5,17 +5,6 @@ * @license The MIT License (MIT); see LICENSE.txt */ -.ve-ui-mwLinkAnnotationInspector-linkTypeSelect .oo-ui-buttonOptionWidget .oo-ui-buttonElement-button { - height: auto; - margin: 0; - padding: 0 0.5em; - color: #888; -} - -.ve-ui-mwLinkAnnotationInspector-linkTypeSelect .oo-ui-optionWidget-selected .oo-ui-buttonElement-button { - background: #eee; -} - -.ve-ui-mwLinkAnnotationInspector .oo-ui-iconElement-icon { - opacity: 0.4; +.ve-ui-mwLinkAnnotationInspector-linkTypeSelect { + margin: -0.75em -0.75em 0.75em -0.75em; } \ No newline at end of file -- To view, visit https://gerrit.wikimedia.org/r/208619 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5d949bf69b0721b288c37c1f12210b0265658eb5 Gerrit-PatchSet: 6 Gerrit-Project: mediawiki/extensions/VisualEditor Gerrit-Branch: master Gerrit-Owner: Esanders <esand...@wikimedia.org> Gerrit-Reviewer: Esanders <esand...@wikimedia.org> Gerrit-Reviewer: Trevor Parscal <tpars...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits