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

Remove alert box on /apply #938

Merged
merged 15 commits into from
Feb 12, 2024
Merged

Remove alert box on /apply #938

merged 15 commits into from
Feb 12, 2024

Conversation

drew-usds
Copy link
Contributor

No description provided.

TomNUSDS and others added 14 commits January 12, 2024 13:52
Merging blog support into main/master.
* Update carousel-projects.html
* Make project carousels more like blog
- Added `carousel_show` and `carousel_image_alt_text` to the markdown "liquid" header.
- Add some comments to this header.
- Fix some line wrapping issues in these headers that may have been causing issues
TODO
 - the `carousel_image_alt_text` is just set to  `"Project photo"` for now. We should customize it. (issue created)
 - Update documentation (issue created)
- Was trying to use Ruby version 3.2.2 but pushing to production failed because the github docker image doesn't support it yet. The newest support version listed is 3.1.3 so trying that.
- Noticed the `Gemlock.lock`'s `PLATFORMS` got trimmed down. Doing experiment to see if it matters.
* Update Gemfile.lock PLATFORMS experiment failed. Reset to old value

* Update deploy.yml Try bumping cache-version to force 3.1.3
* Fix one broken link in blog
* Read application.css
Apply form needs this file. Not able to test on staging site.
- Update twitter logo to X
- Remove Facebook link
- Bring up-to-date with main
* Fix minor Apply iframe height issue.

In some situations, the Apply iframe is too short and it gets a nested scrollbar. It's most obvious when changing the "zoom" size or if viewing on mobile.

Fix changes:
- size to use `rem/em` vs `px` for browser zoom levels
- uses "mobile first" breakpoint that's taller for mobile and shorter for desktop.

No good way to test in localhost and staging because the iframe won't load. Attempted to test by applying new sizes to the production site via the debugger.

Rebuilding the css is problematic. Still must delete the style.css file and run the *release build* to get the correct output (not the `serve` or `start).

* Add breakpoints for tablet and desktop

- `var frameResizedHeight = parseInt(height) + 25;` => `const frameResizedHeight = parseInt(height) + 307;` for the salesforce callback.
- Adjusted reactive css for Apply form to handle case where user resizes the browser. It's set to max height when both the "how did you hear about usds" and "portfolio = design cop" are selected in the form.
* add 6 articles

* Update news_articles.yml

* add press logos

* add press logo
* initial draft

* add URLs

* typo

* add AI blog image

* add folder

* image file path

* edit card summary

* Update _news-and-blog/2024-01-29-a-call-to-service-for-talent-in-the-federal-government.md

Co-authored-by: Drew Gardner <57154955+drew-usds@users.noreply.github.com>

* Update _news-and-blog/2024-01-29-a-call-to-service-for-talent-in-the-federal-government.md

Co-authored-by: Drew Gardner <57154955+drew-usds@users.noreply.github.com>

---------

Co-authored-by: Drew Gardner <57154955+drew-usds@users.noreply.github.com>
Remove scheduled maintenance alert on /apply
Copy link

⚠️ Visual differences detected.
For details, click the green checkmark above.
If your PR makes intentional visual changes, update the base images by running the Visual regression testing Action.

Copy link
Contributor

@TomNUSDS TomNUSDS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@drew-usds drew-usds merged commit 09ea146 into staging Feb 12, 2024
2 checks passed
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.

None yet

3 participants