Commit e7faeda9 authored by Yogi's avatar Yogi Committed by David O'Regan

Migrate old button classes to our Pajamas style GitLab button

Add gl-button to new project button

Add gl-button to send report button

Add gl-button to Buy additional minutes

Add gl-button to project edit and delete button

Add gl-button to Save changes button

Add gl-button to js-filepicker-button

Add more gl-button

Add more gl-button

Add more gl-button

Add more gl-button

Add more gl-button

Add gl-button to Create/Save milestone

Add Changelog

Add gl-button to add list button

Add gl-button to create label button

Add gl-button to View epics list button

Add gl-buttons in bulk update

Apply 1 suggestion(s) to 1 file(s)

Apply 1 suggestion(s) to 1 file(s)

Move to GlButton component

Apply suggestion 51308#note_486380698

Apply 1 suggestion(s) to 1 file(s)
parent 4bbbe277
...@@ -25,4 +25,4 @@ ...@@ -25,4 +25,4 @@
= _("Explain the problem. If appropriate, provide a link to the relevant issue or comment.") = _("Explain the problem. If appropriate, provide a link to the relevant issue or comment.")
.form-actions .form-actions
= f.submit _("Send report"), class: "btn btn-success" = f.submit _("Send report"), class: "gl-button btn btn-success"
...@@ -33,5 +33,5 @@ ...@@ -33,5 +33,5 @@
= visibility_level_icon(group.visibility_level) = visibility_level_icon(group.visibility_level)
.controls.gl-flex-shrink-0.gl-ml-5 .controls.gl-flex-shrink-0.gl-ml-5
= link_to _('Edit'), admin_group_edit_path(group), id: "edit_#{dom_id(group)}", class: 'btn' = link_to _('Edit'), admin_group_edit_path(group), id: "edit_#{dom_id(group)}", class: 'gl-button btn'
= link_to _('Delete'), [:admin, group], data: { confirm: _("Are you sure you want to remove %{group_name}?") % { group_name: group.name } }, method: :delete, class: 'gl-button btn btn-danger' = link_to _('Delete'), [:admin, group], data: { confirm: _("Are you sure you want to remove %{group_name}?") % { group_name: group.name } }, method: :delete, class: 'gl-button btn btn-danger'
...@@ -4,8 +4,8 @@ ...@@ -4,8 +4,8 @@
- @projects.each_with_index do |project| - @projects.each_with_index do |project|
%li.project-row{ class: ('no-description' if project.description.blank?) } %li.project-row{ class: ('no-description' if project.description.blank?) }
.controls .controls
= link_to 'Edit', edit_project_path(project), id: "edit_#{dom_id(project)}", class: "btn" = link_to 'Edit', edit_project_path(project), id: "edit_#{dom_id(project)}", class: "gl-button btn"
%button.delete-project-button.btn.btn-danger{ data: { delete_project_url: admin_project_path(project), project_name: project.name } } %button.delete-project-button.gl-button.btn.btn-danger{ data: { delete_project_url: admin_project_path(project), project_name: project.name } }
= s_('AdminProjects|Delete') = s_('AdminProjects|Delete')
.stats .stats
......
...@@ -32,7 +32,7 @@ ...@@ -32,7 +32,7 @@
= image_tag avatar_icon_for_user(@user, 160), alt: '', class: 'avatar s160' = image_tag avatar_icon_for_user(@user, 160), alt: '', class: 'avatar s160'
%h5.gl-mt-0= s_("Profiles|Upload new avatar") %h5.gl-mt-0= s_("Profiles|Upload new avatar")
.gl-mt-2.gl-mb-3 .gl-mt-2.gl-mb-3
%button.btn.js-choose-user-avatar-button{ type: 'button' }= s_("Profiles|Choose file...") %button.gl-button.btn.js-choose-user-avatar-button{ type: 'button' }= s_("Profiles|Choose file...")
%span.avatar-file-name.gl-ml-3.js-avatar-filename= s_("Profiles|No file chosen") %span.avatar-file-name.gl-ml-3.js-avatar-filename= s_("Profiles|No file chosen")
= f.file_field_without_bootstrap :avatar, class: 'js-user-avatar-input hidden', accept: 'image/*' = f.file_field_without_bootstrap :avatar, class: 'js-user-avatar-input hidden', accept: 'image/*'
.form-text.text-muted= s_("Profiles|The maximum file size allowed is 200KB.") .form-text.text-muted= s_("Profiles|The maximum file size allowed is 200KB.")
......
...@@ -25,7 +25,7 @@ ...@@ -25,7 +25,7 @@
.js-project-permissions-form .js-project-permissions-form
- if show_visibility_confirm_modal?(@project) - if show_visibility_confirm_modal?(@project)
= render "visibility_modal" = render "visibility_modal"
= f.submit _('Save changes'), class: "btn btn-success #{('js-confirm-danger' if show_visibility_confirm_modal?(@project))}", data: { qa_selector: 'visibility_features_permissions_save_button', check_field_name: ("project[visibility_level]" if show_visibility_confirm_modal?(@project)), check_compare_value: @project.visibility_level } = f.submit _('Save changes'), class: "gl-button gl-button btn btn-success #{('js-confirm-danger' if show_visibility_confirm_modal?(@project))}", data: { qa_selector: 'visibility_features_permissions_save_button', check_field_name: ("project[visibility_level]" if show_visibility_confirm_modal?(@project)), check_compare_value: @project.visibility_level }
%section.qa-merge-request-settings.rspec-merge-request-settings.settings.merge-requests-feature.no-animate#js-merge-request-settings{ class: [('expanded' if expanded), ('hidden' if @project.project_feature.send(:merge_requests_access_level) == 0)] } %section.qa-merge-request-settings.rspec-merge-request-settings.settings.merge-requests-feature.no-animate#js-merge-request-settings{ class: [('expanded' if expanded), ('hidden' if @project.project_feature.send(:merge_requests_access_level) == 0)] }
.settings-header .settings-header
...@@ -39,7 +39,7 @@ ...@@ -39,7 +39,7 @@
= form_for @project, remote: true, html: { multipart: true, class: "merge-request-settings-form js-mr-settings-form" }, authenticity_token: true do |f| = form_for @project, remote: true, html: { multipart: true, class: "merge-request-settings-form js-mr-settings-form" }, authenticity_token: true do |f|
%input{ name: 'update_section', type: 'hidden', value: 'js-merge-request-settings' } %input{ name: 'update_section', type: 'hidden', value: 'js-merge-request-settings' }
= render 'projects/merge_request_settings', form: f = render 'projects/merge_request_settings', form: f
= f.submit _('Save changes'), class: "btn btn-succes qa-save-merge-request-changes rspec-save-merge-request-changes" = f.submit _('Save changes'), class: "gl-button btn btn-succes qa-save-merge-request-changes rspec-save-merge-request-changes"
= render_if_exists 'projects/merge_request_approvals_settings', expanded: expanded = render_if_exists 'projects/merge_request_approvals_settings', expanded: expanded
...@@ -70,7 +70,7 @@ ...@@ -70,7 +70,7 @@
%h4= _('Housekeeping') %h4= _('Housekeeping')
%p= _('Runs a number of housekeeping tasks within the current repository, such as compressing file revisions and removing unreachable objects.') %p= _('Runs a number of housekeeping tasks within the current repository, such as compressing file revisions and removing unreachable objects.')
= link_to _('Run housekeeping'), housekeeping_project_path(@project), = link_to _('Run housekeeping'), housekeeping_project_path(@project),
method: :post, class: "btn btn-default" method: :post, class: "gl-button btn btn-default"
= render 'export', project: @project = render 'export', project: @project
...@@ -92,7 +92,7 @@ ...@@ -92,7 +92,7 @@
%li= _('You will need to update your local repositories to point to the new location.') %li= _('You will need to update your local repositories to point to the new location.')
- if @project.deployment_platform.present? - if @project.deployment_platform.present?
%li= _('Your deployment services will be broken, you will need to manually fix the services after renaming.') %li= _('Your deployment services will be broken, you will need to manually fix the services after renaming.')
= f.submit _('Change path'), class: "btn btn-warning qa-change-path-button" = f.submit _('Change path'), class: "gl-button btn btn-warning qa-change-path-button"
= render 'transfer', project: @project = render 'transfer', project: @project
......
...@@ -21,8 +21,8 @@ ...@@ -21,8 +21,8 @@
.form-actions .form-actions
- if @milestone.new_record? - if @milestone.new_record?
= f.submit _('Create milestone'), class: 'btn-success btn', data: { qa_selector: 'create_milestone_button' } = f.submit _('Create milestone'), class: 'gl-button btn-success btn', data: { qa_selector: 'create_milestone_button' }
= link_to _('Cancel'), project_milestones_path(@project), class: 'gl-button btn btn-cancel' = link_to _('Cancel'), project_milestones_path(@project), class: 'gl-button btn btn-cancel'
- else - else
= f.submit _('Save changes'), class: 'btn-success btn' = f.submit _('Save changes'), class: 'gl-button btn-success btn'
= link_to _('Cancel'), project_milestone_path(@project, @milestone), class: 'gl-button btn btn-cancel' = link_to _('Cancel'), project_milestone_path(@project, @milestone), class: 'gl-button btn btn-cancel'
...@@ -7,4 +7,4 @@ ...@@ -7,4 +7,4 @@
.gl-alert-body .gl-alert-body
= s_('AlertSettings|You can now set up alert endpoints for manually configured Prometheus instances in the Alerts section on the Operations settings page. Alert endpoint fields on this page have been deprecated.') = s_('AlertSettings|You can now set up alert endpoints for manually configured Prometheus instances in the Alerts section on the Operations settings page. Alert endpoint fields on this page have been deprecated.')
.gl-alert-actions .gl-alert-actions
= link_to _('Visit settings page'), project_settings_operations_path(@project, anchor: 'js-alert-management-settings'), class: 'btn gl-alert-action btn-info gl-button' = link_to _('Visit settings page'), project_settings_operations_path(@project, anchor: 'js-alert-management-settings'), class: 'gl-button btn gl-alert-action btn-info'
...@@ -10,9 +10,9 @@ ...@@ -10,9 +10,9 @@
%p= _("Unarchiving the project will restore people's ability to make changes to it. The repository can be committed to, and issues, comments, and other entities can be created. %{strong_start}Once active, this project shows up in the search and on the dashboard.%{strong_end}").html_safe % { strong_start: '<strong>'.html_safe, strong_end: '</strong>'.html_safe } %p= _("Unarchiving the project will restore people's ability to make changes to it. The repository can be committed to, and issues, comments, and other entities can be created. %{strong_start}Once active, this project shows up in the search and on the dashboard.%{strong_end}").html_safe % { strong_start: '<strong>'.html_safe, strong_end: '</strong>'.html_safe }
= link_to _('Unarchive project'), unarchive_project_path(@project), = link_to _('Unarchive project'), unarchive_project_path(@project),
data: { confirm: _("Are you sure that you want to unarchive this project?"), qa_selector: 'unarchive_project_link' }, data: { confirm: _("Are you sure that you want to unarchive this project?"), qa_selector: 'unarchive_project_link' },
method: :post, class: "btn btn-success" method: :post, class: "gl-button btn btn-success"
- else - else
%p= _("Archiving the project will make it entirely read only. It is hidden from the dashboard and doesn't show up in searches. %{strong_start}The repository cannot be committed to, and no issues, comments, or other entities can be created.%{strong_end}").html_safe % { strong_start: '<strong>'.html_safe, strong_end: '</strong>'.html_safe } %p= _("Archiving the project will make it entirely read only. It is hidden from the dashboard and doesn't show up in searches. %{strong_start}The repository cannot be committed to, and no issues, comments, or other entities can be created.%{strong_end}").html_safe % { strong_start: '<strong>'.html_safe, strong_end: '</strong>'.html_safe }
= link_to _('Archive project'), archive_project_path(@project), = link_to _('Archive project'), archive_project_path(@project),
data: { confirm: _("Are you sure that you want to archive this project?"), qa_selector: 'archive_project_link' }, data: { confirm: _("Are you sure that you want to archive this project?"), qa_selector: 'archive_project_link' },
method: :post, class: "btn btn-warning" method: :post, class: "gl-button btn btn-warning"
...@@ -40,4 +40,4 @@ ...@@ -40,4 +40,4 @@
%hr %hr
= link_to _('Remove avatar'), project_avatar_path(@project), data: { confirm: _('Avatar will be removed. Are you sure?')}, method: :delete, class: 'btn btn-link' = link_to _('Remove avatar'), project_avatar_path(@project), data: { confirm: _('Avatar will be removed. Are you sure?')}, method: :delete, class: 'btn btn-link'
= f.submit _('Save changes'), class: "btn btn-success mt-4 qa-save-naming-topics-avatar-button" = f.submit _('Save changes'), class: "gl-button btn btn-success gl-mt-6 qa-save-naming-topics-avatar-button"
...@@ -10,7 +10,7 @@ ...@@ -10,7 +10,7 @@
.gl-alert-body .gl-alert-body
= _('Webhooks have moved. They can now be found under the Settings menu.') = _('Webhooks have moved. They can now be found under the Settings menu.')
.gl-alert-actions .gl-alert-actions
= link_to _('Go to Webhooks'), project_hooks_path(@project), class: 'btn gl-alert-action btn-info new-gl-button' = link_to _('Go to Webhooks'), project_hooks_path(@project), class: 'gl-button btn gl-alert-action btn-info'
%h4= _('Integrations') %h4= _('Integrations')
- integrations_link_start = '<a href="%{url}">'.html_safe % { url: help_page_url('user/project/integrations/overview') } - integrations_link_start = '<a href="%{url}">'.html_safe % { url: help_page_url('user/project/integrations/overview') }
......
- classes = local_assigns.fetch(:classes, '') - classes = local_assigns.fetch(:classes, '')
%span.js-filepicker %span.js-filepicker
%button.btn.js-filepicker-button{ type: 'button', class: classes }= _("Choose file…") %button.gl-button.btn.js-filepicker-button{ type: 'button', class: classes }= _("Choose file…")
%span.file_name.js-filepicker-filename= _("No file chosen") %span.file_name.js-filepicker-filename= _("No file chosen")
= f.file_field field, class: "js-filepicker-input hidden" = f.file_field field, class: "js-filepicker-input hidden"
- if help_text.present? - if help_text.present?
......
...@@ -46,7 +46,7 @@ ...@@ -46,7 +46,7 @@
%button.js-unsubscribe-button.label-subscribe-button.btn.btn-default{ class: ('hidden' if status.unsubscribed?), data: { url: toggle_subscription_path, toggle: 'tooltip' }, title: tooltip_title } %button.js-unsubscribe-button.label-subscribe-button.btn.btn-default{ class: ('hidden' if status.unsubscribed?), data: { url: toggle_subscription_path, toggle: 'tooltip' }, title: tooltip_title }
%span= _('Unsubscribe') %span= _('Unsubscribe')
.dropdown.dropdown-group-label{ class: ('hidden' unless status.unsubscribed?) } .dropdown.dropdown-group-label{ class: ('hidden' unless status.unsubscribed?) }
%button.label-subscribe-button.btn.btn-default{ data: { toggle: 'dropdown' } } %button.gl-button.label-subscribe-button.btn.btn-default{ data: { toggle: 'dropdown' } }
%span %span
= _('Subscribe') = _('Subscribe')
= sprite_icon('chevron-down') = sprite_icon('chevron-down')
...@@ -59,7 +59,7 @@ ...@@ -59,7 +59,7 @@
%button.js-subscribe-button.js-group-level.label-subscribe-button.btn.btn-default{ class: ('hidden' unless status.unsubscribed?), data: { status: status, url: toggle_subscription_group_label_path(label.group, label) } } %button.js-subscribe-button.js-group-level.label-subscribe-button.btn.btn-default{ class: ('hidden' unless status.unsubscribed?), data: { status: status, url: toggle_subscription_group_label_path(label.group, label) } }
%span= _('Subscribe at group level') %span= _('Subscribe at group level')
- else - else
%button.js-subscribe-button.label-subscribe-button.btn.btn-default{ data: { status: status, url: toggle_subscription_path, toggle: 'tooltip' }, title: tooltip_title } %button.gl-button.js-subscribe-button.label-subscribe-button.btn.btn-default{ data: { status: status, url: toggle_subscription_path, toggle: 'tooltip' }, title: tooltip_title }
%span= label_subscription_toggle_button_text(label, @project) %span= label_subscription_toggle_button_text(label, @project)
= render 'shared/delete_label_modal', label: label = render 'shared/delete_label_modal', label: label
...@@ -12,9 +12,9 @@ ...@@ -12,9 +12,9 @@
%p= current_user_empty_message_description %p= current_user_empty_message_description
- if secondary_button_link.present? - if secondary_button_link.present?
= link_to secondary_button_label, secondary_button_link, class: 'btn btn-success btn-inverted' = link_to secondary_button_label, secondary_button_link, class: 'gl-button btn btn-success btn-inverted'
- if primary_button_link.present? - if primary_button_link.present?
= link_to primary_button_label, primary_button_link, class: 'btn btn-success' = link_to primary_button_label, primary_button_link, class: 'gl-button btn btn-success'
- else - else
%h5= visitor_empty_message %h5= visitor_empty_message
.dropdown.gl-ml-3#js-add-list .dropdown.gl-ml-3#js-add-list
%button.btn.btn-success.btn-inverted.js-new-board-list{ type: "button", data: board_list_data } %button.gl-button.btn.btn-success.btn-inverted.js-new-board-list{ type: "button", data: board_list_data }
Add list Add list
.dropdown-menu.dropdown-extended-height.dropdown-menu-paging.dropdown-menu-right.dropdown-menu-issues-board-new.dropdown-menu-selectable.js-tab-container-labels .dropdown-menu.dropdown-extended-height.dropdown-menu-paging.dropdown-menu-right.dropdown-menu-issues-board-new.dropdown-menu-selectable.js-tab-container-labels
= render partial: "shared/issuable/label_page_default", locals: { show_footer: true, show_create: true, show_boards_content: true, title: "Add list" } = render partial: "shared/issuable/label_page_default", locals: { show_footer: true, show_create: true, show_boards_content: true, title: "Add list" }
......
...@@ -7,8 +7,8 @@ ...@@ -7,8 +7,8 @@
= form_tag [:bulk_update, @project, type], method: :post, class: "bulk-update" do = form_tag [:bulk_update, @project, type], method: :post, class: "bulk-update" do
.block.issuable-sidebar-header .block.issuable-sidebar-header
.filter-item.inline.update-issues-btn.float-left .filter-item.inline.update-issues-btn.float-left
= button_tag _('Update all'), class: "btn update-selected-issues btn-info", disabled: true = button_tag _('Update all'), class: "gl-button btn update-selected-issues btn-info", disabled: true
= button_tag _('Cancel'), class: "btn btn-default js-bulk-update-menu-hide float-right" = button_tag _('Cancel'), class: "gl-button btn btn-default js-bulk-update-menu-hide float-right"
- if params[:state] != 'merged' - if params[:state] != 'merged'
.block .block
.title .title
......
...@@ -19,7 +19,7 @@ ...@@ -19,7 +19,7 @@
%input.js-add-list{ type: "checkbox", name: "add_list", checked: add_list } %input.js-add-list{ type: "checkbox", name: "add_list", checked: add_list }
%span= _('Add list') %span= _('Add list')
.clearfix .clearfix
%button.btn.btn-primary.float-left.js-new-label-btn{ type: "button" } %button.gl-button.btn.btn-success.float-left.js-new-label-btn{ type: "button" }
= _('Create') = _('Create')
%button.btn.btn-default.float-right.js-cancel-label-btn{ type: "button" } %button.gl-button.btn.btn-default.float-right.js-cancel-label-btn{ type: "button" }
= _('Cancel') = _('Cancel')
---
title: Migrate old button classes to our Pajamas style GitLab button in multiple areas
merge_request: 51826
author: Yogi (@yo)
type: other
<script> <script>
/* eslint-disable vue/no-v-html */ /* eslint-disable vue/no-v-html */
import { GlButton } from '@gitlab/ui';
import { s__, sprintf } from '~/locale'; import { s__, sprintf } from '~/locale';
import { dateInWords } from '~/lib/utils/datetime_utility'; import { dateInWords } from '~/lib/utils/datetime_utility';
...@@ -9,6 +10,9 @@ import { emptyStateDefault, emptyStateWithFilters } from '../constants'; ...@@ -9,6 +10,9 @@ import { emptyStateDefault, emptyStateWithFilters } from '../constants';
import initEpicCreate from '../../epic/epic_bundle'; import initEpicCreate from '../../epic/epic_bundle';
export default { export default {
components: {
GlButton,
},
mixins: [CommonMixin], mixins: [CommonMixin],
props: { props: {
presetType: { presetType: {
...@@ -139,9 +143,9 @@ export default { ...@@ -139,9 +143,9 @@ export default {
id="epic-create-root" id="epic-create-root"
:data-endpoint="newEpicEndpoint" :data-endpoint="newEpicEndpoint"
></div> ></div>
<a :title="__('List')" :href="newEpicEndpoint" class="btn btn-default"> <gl-button :title="__('List')" :href="newEpicEndpoint">{{
<span>{{ s__('View epics list') }}</span> s__('View epics list')
</a> }}</gl-button>
</div> </div>
</div> </div>
</div> </div>
......
...@@ -9,7 +9,7 @@ ...@@ -9,7 +9,7 @@
= link_to s_('UsageQuota|Buy additional minutes'), = link_to s_('UsageQuota|Buy additional minutes'),
EE::SUBSCRIPTIONS_MORE_MINUTES_URL, EE::SUBSCRIPTIONS_MORE_MINUTES_URL,
target: '_blank', target: '_blank',
class: 'btn btn-success float-right', class: 'gl-button btn btn-success float-right',
data: { track_event: 'click_buy_ci_minutes', track_label: namespace.actual_plan_name, track_property: 'pipeline_quota_page' } data: { track_event: 'click_buy_ci_minutes', track_label: namespace.actual_plan_name, track_property: 'pipeline_quota_page' }
.row .row
......
...@@ -8,8 +8,8 @@ ...@@ -8,8 +8,8 @@
= form_tag [:bulk_update, group, type], method: :post, class: "bulk-update" do = form_tag [:bulk_update, group, type], method: :post, class: "bulk-update" do
.block.issuable-sidebar-header .block.issuable-sidebar-header
.filter-item.inline.update-issues-btn.float-left .filter-item.inline.update-issues-btn.float-left
= button_tag _('Update all'), class: "btn update-selected-issues btn-info", disabled: true = button_tag _('Update all'), class: "gl-button btn update-selected-issues btn-info", disabled: true
= button_tag _('Cancel'), class: "btn btn-default js-bulk-update-menu-hide float-right" = button_tag _('Cancel'), class: "gl-button btn btn-default js-bulk-update-menu-hide float-right"
- if epic_bulk_edit_flag - if epic_bulk_edit_flag
.block .block
.title .title
......
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