Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FIX: Team listing elements #377

Closed
wants to merge 22 commits into from
Closed

FIX: Team listing elements #377

wants to merge 22 commits into from

Conversation

iseries
Copy link
Contributor

@iseries iseries commented Nov 23, 2020

  • toggle boxes for profile details are behind the footer and hide informations
  • long url break the layout

image

regniets and others added 22 commits September 30, 2020 10:27
- Removed old SEO Mixins as they are being added to Neos.Neos:Document
anyway
- Made noindex/nofollow editable for correct language handling
- Moved metadata back to seo tab for coherence
Added Validators to countup, solves issue #327
Added translation for choiceButton; refactored to afx
Refactoring of CaseStudies/Showcases and Service Provider Listing
Bugfix: Add showcase listing on service provider page
* FEATURE: Switch to Matomo for tracking

After deployment the following steps need to be taken:

* Add opt-out element to legal page
* Adjust text on legal page
* Make sure environment variable for auth token is set
* Check all tags that they work properly

Co-authored-by: Stefan Regniet <s.regniet@techdivision.com>
Bugfix: Added maximumWidth and async for caselisting images
Lazyload bugfix: Added vanilla-lazyload for lazyloading
remove overflow hidden in .stage because it cuts .team-listing-profile boxes near the footer section
Add overflow:hidden to the body tag to avoid horizontal scrolling when using oversized elements like .team-listing-profile
Long urls causing layout problems due to overflow in .team-listing-profile. Better wrap it.
@kdambekalns kdambekalns force-pushed the master branch 2 times, most recently from edca575 to 50a939d Compare March 12, 2021 14:07
@Sebobo
Copy link
Member

Sebobo commented Sep 9, 2021

Thank you @iseries, I cherrypicked the change into master manually as this PR was a bit "conflicted"

@Sebobo Sebobo closed this Sep 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants