Skip to content
This repository has been archived by the owner on May 18, 2023. It is now read-only.

End of Kabanero Website Updates #379

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

kSee04
Copy link
Contributor

@kSee04 kSee04 commented Apr 9, 2021

What was fixed? (Issue # and description of fix)

This PR has changes for the website related to end of Kabanero UI changes. I'm not sure if there's an issue for this somewhere so I'll paste in the email Anne sent with requirements.

Screen Shot 2021-04-09 at 1 54 53 PM

Per item 1 in the email here is a screenshot of the banner
Screen Shot 2021-04-09 at 3 49 08 PM

the learn more button links directly to the adois blog post which per item 2 in the email doesn't have the hotlink icons to twitter and slack and below is a screenshot of that
Screen Shot 2021-04-09 at 3 49 18 PM

@anneqm
Copy link
Member

anneqm commented Apr 9, 2021

can we also remove the icons for twitter and slack on the red banner? Is that possible?

Copy link
Member

@anneqm anneqm left a comment

Choose a reason for hiding this comment

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

can we also remove the slack icon and twitter icon on the red banner bar? Else looks good to me

@kSee04
Copy link
Contributor Author

kSee04 commented Apr 9, 2021

@anneqm I pushed up the changes & updated the pictures at the top of the PR.

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.

2 participants