Skip to content
This repository has been archived by the owner on Oct 13, 2021. It is now read-only.

Add responsive styles to announcement subscribe button #749

Merged
merged 3 commits into from
Sep 26, 2019
Merged

Conversation

WilHall
Copy link
Contributor

@WilHall WilHall commented Sep 21, 2019

Fixes #678.

Adds responsive styles to the announcement subscribe button so that it doesn't squish the author.

Portrait Image 2019-09-21 at 4 00 17 PM
Landscape Image 2019-09-21 at 4 00 28 PM

@WilHall WilHall self-assigned this Sep 21, 2019
assets/css/components/_announcements.scss Outdated Show resolved Hide resolved
lib/constable_web/templates/announcement/show.html.eex Outdated Show resolved Hide resolved
@WilHall
Copy link
Contributor Author

WilHall commented Sep 25, 2019

@tysongach All set with this PR but have you noticed that a test started failing on CI (see this Circle build)? Same for #748, #750, and #751. Oddly I cannot reproduce it locally - that test passes fine. Any thoughts on what that's all about?

@WilHall WilHall merged commit c834fdd into master Sep 26, 2019
@WilHall WilHall deleted the author-squish branch September 26, 2019 00:07
@tysongach
Copy link
Contributor

@WilHall Seem like a flaky test maybe - some of those PRs you linked to are now passing, others still failing. It looks like maybe a Markdown parser issue? It’s perhaps escaping HTML differently?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Author info gets squished by “subscribe to thread” button on small screens
2 participants