-
Notifications
You must be signed in to change notification settings - Fork 0
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
Updating a style to be applied to the federated site. #435
Conversation
JLG 12012023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@goynejennifer is there a specific reason why this is set to null?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure why this is set to Null did I set this to null. I probably should be set to a value.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It came from your merge request. So I guess its you.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LOL. As far as I know it's just extra files the Visual Studio creates but maybe I don't need to check in just to keep locally.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@goynejennifer I am not sure if this is required for the project. I see your local path so not sure how this is used?
JLG 12212023
Preview branch generated at https://JLG-DevUpdateFederatedStyles.d234tdmg9icdwk.amplifyapp.com |
JLG 12012023
Updating Style in FeatureItems.scss to get the circles in the federated to work properly
JIRA Ticket: (link) (:star:)
What does this Pull Request do? (:star:)
This pull request should fix the issue with the 3 circles that are on the federated preproduction site.
What's the changes? (:star:)
The FeatureItems.scss line 19 was commented out to get the three circles on the federated site working.
Example:
Updated FeatureItems.scss, line 19, commented out the height because it was messing up the circles.
How should this be tested?
To test this out push the updated .scss style file to the preproduction federated website to test if the change does work on the federated website. https://federated-pprd.dlp.cloud.lib.vt.edu/
A description of what steps someone could take to:
None
Additional Notes:
Any additional information that you think would be helpful when reviewing this PR.
Example:
This was branched off the main dev branch.
Interested parties
Tag (@ mention) interested parties
(:star:) Required fields