Skip to content

Commit

Permalink
bug/691-css-bughunt (#725)
Browse files Browse the repository at this point in the history
* Add spacing to profile boxes

* Fix spacing of profile

* Reverse removing of flash styling

* Add info message to profile

* Improve no rated skills sentences

---------

Co-authored-by: Robin Steiner <[email protected]>
  • Loading branch information
ManuelMoeri and Robin481 authored Jun 11, 2024
1 parent 3e6a5c0 commit 6324796
Show file tree
Hide file tree
Showing 7 changed files with 13 additions and 6 deletions.
7 changes: 4 additions & 3 deletions app/views/layouts/application.html.haml
Original file line number Diff line number Diff line change
Expand Up @@ -69,8 +69,9 @@
%a.nav-link.cursor-pointer.ps-2.pe-2{href: skills_path} Skillset
%div.container-fluid
%div.row.d-flex.justify-content-center
#flash= render partial: 'layouts/flash', collection: [:notice, :alert], as: :level
%span
= yield :actions
= render partial: 'layouts/flash', collection: [:notice, :alert], as: :level
- content_for(:actions) do
%span
= yield :actions
= content_for?(:content) ? yield(:content) : yield
= turbo_frame_tag "remote_modal", target: "_top"
2 changes: 1 addition & 1 deletion app/views/people/_core_competences.html.haml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
%div.border.border-secondary-subtle.border-1.d-flex.flex-column#core-competences{"data-scroll-target": "scrollItem"}
%div.border.border-secondary-subtle.border-1.d-flex.flex-column.mb-3#core-competences{"data-scroll-target": "scrollItem"}
%div.profile-header.mw-100.border-bottom
Kernkompetenzen
%div.d-flex.flex-column.ms-5.mt-3.mb-3
Expand Down
4 changes: 4 additions & 0 deletions app/views/people/_cv.html.haml
Original file line number Diff line number Diff line change
@@ -1,6 +1,10 @@
= render 'scroll_to_menu', items: %w[personal-data core-competences educations advanced-trainings activities projects], translate: true do
= link_to image_tag("export.svg") + "Export", export_cv_person_path(@person), data: { turbo_frame: "remote_modal" }
%div.w-100{data: { "scroll-target": "parent"}}
- if @person.people_skills.count == 0
%span
%p.alert.alert-info= t('profile.no_skills_rated_msg')
= render('profile')
= render('core_competences')
= render('people/person_relations/index', list: @person.educations.list, id: "educations")
Expand Down
2 changes: 1 addition & 1 deletion app/views/people/_profile.html.haml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
%div#personal-data{"data-scroll-target": "scrollItem"}
%div.mb-3#personal-data{"data-scroll-target": "scrollItem"}
%div.profile-header.mw-100.border-bottom
Personalien
%div.mt-4
Expand Down
2 changes: 1 addition & 1 deletion app/views/people/person_relations/_index.html.haml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
%div.border.border-secondary-subtle.border-1.d-flex.flex-column{id: id, "data-scroll-target": "scrollItem"}
%div.border.border-secondary-subtle.border-1.d-flex.flex-column.mb-3{id: id, "data-scroll-target": "scrollItem"}
%div.profile-header.mw-100.border-bottom
= "#{t "activerecord.models.#{name_of_obj(list)}"} (#{list.count})"
%div.d-flex.flex-column.ms-5.mt-3.mb-3
Expand Down
1 change: 1 addition & 0 deletions config/locales/de.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ de:
activities: Stationen
projects: Projekte
updated_at: Zuletzt bearbeitet
no_skills_rated_msg: Dieses Profil hat noch keine bewerteten Skills. Gehe zum Skills Tab und bewerte die für dich relevanten Skills.
attributes:
year_from: Jahr von
year_to: Jahr bis
Expand Down
1 change: 1 addition & 0 deletions config/locales/en.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ en:
activities: Activities
projects: Projects
updated_at: Zuletzt bearbeitet
no_skills_rated_msg: This profile doesn't have any rated skills yet, head to the Skills Tab and rate some of the skills relevant to you.
skills:
header:
all: All
Expand Down

0 comments on commit 6324796

Please sign in to comment.