Commit b7a31605 authored by Justin Mandell's avatar Justin Mandell Committed by Jose Ivan Vargas

Swap primary for secondary buttons in Security Configure page

parent 4efdb904
......@@ -31,13 +31,12 @@ export default {
const button = this.enabled
? {
text: this.$options.i18n.configureFeature,
category: 'secondary',
}
: {
text: this.$options.i18n.enableFeature,
category: 'primary',
};
button.category = 'secondary';
button.text = sprintf(button.text, { feature: this.shortName });
return button;
......@@ -126,7 +125,7 @@ export default {
v-else-if="showManageViaMr"
:feature="feature"
variant="confirm"
category="primary"
category="secondary"
class="gl-mt-5"
:data-qa-selector="`${feature.type}_mr_button`"
@error="onError"
......
......@@ -50,7 +50,7 @@ describe('FeatureCard component', () => {
expect(enableLinks.exists()).toBe(expectEnableAction);
if (expectEnableAction) {
expect(enableLinks).toHaveLength(1);
expect(enableLinks.at(0).props('category')).toBe('primary');
expect(enableLinks.at(0).props('category')).toBe('secondary');
}
const configureLinks = findConfigureLinks();
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment