Skip to content

Commit

Permalink
Merge pull request #10926 from saraycp/adjust-margin
Browse files Browse the repository at this point in the history
Adjust margin and padding on edit links
  • Loading branch information
hennevogel committed Mar 18, 2021
2 parents 0d4a189 + c81bfac commit e77b9b4
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 6 deletions.
9 changes: 5 additions & 4 deletions src/api/app/views/webui/package/_basic_info.html.haml
@@ -1,7 +1,7 @@
.editing-form.d-none
= render partial: 'edit', locals: { package: package, project: package.project }

.basic-info.mb-3
.basic-info
%h3#package-title
= package.title.presence || package.name
- if package.url.present?
Expand All @@ -13,9 +13,10 @@
= render partial: 'webui/shared/collapsible_text', locals: { text: package.description }

- if policy(package).update?
= link_to('javascript:void(0);', id: 'toggle-in-place-editing', class: 'nav-link', title: 'Edit') do
%i.fas.fa-edit
%span.nav-item-name Edit
.pt-4
= link_to('javascript:void(0);', id: 'toggle-in-place-editing', class: 'nav-link pl-0', title: 'Edit') do
%i.fas.fa-edit
%span.nav-item-name Edit

:javascript
$('#toggle-in-place-editing').on('click', function () {
Expand Down
2 changes: 1 addition & 1 deletion src/api/app/views/webui/project/_basic_info.html.haml
Expand Up @@ -14,6 +14,6 @@

- if policy(project).update?
.pt-4
= link_to(edit_project_path(project), class: 'nav-link', remote: true, title: 'Edit') do
= link_to(edit_project_path(project), class: 'nav-link pl-0', remote: true, title: 'Edit') do
%i.fas.fa-edit.text-secondary
%span.nav-item-name Edit
Expand Up @@ -6,7 +6,7 @@
- if configuration.accounts_editable?(user)
.d-flex.flex-row-reverse
= link_to('javascript:void(0);', id: 'toggle-in-place-editing', class: 'nav-link', remote: true, title: 'Edit Your Account') do
%i.fas.mr-2.fa-user-edit
%i.fas.fa-user-edit
%span.nav-item-name Edit

.row.mb-3.mb-md-0
Expand Down

0 comments on commit e77b9b4

Please sign in to comment.