Skip to content

AI Launch 2023 - Add TeachAI blog callout to /ai page - #51448

Merged
kelbyhawn merged 10 commits into
stagingfrom
ai-page-teachai-blog-blurb
Apr 25, 2023
Merged

AI Launch 2023 - Add TeachAI blog callout to /ai page#51448
kelbyhawn merged 10 commits into
stagingfrom
ai-page-teachai-blog-blurb

Conversation

@kelbyhawn

@kelbyhawn kelbyhawn commented Apr 20, 2023

Copy link
Copy Markdown
Contributor

Add a callout blurb and button in the hero banner on https://code.org/ai to support the launch of TeachAI.

Shown using the teach-ai-launch-2023 DCDO flag.

Related PRs:

Jira ticket: ACQ-546


Screen.Recording.2023-04-25.at.8.53.15.AM.mov

@kelbyhawn
kelbyhawn requested a review from a team as a code owner April 20, 2023 22:41
page will be updated soon w/ microsite
@kelbyhawn
kelbyhawn requested review from a team and dmcavoy April 25, 2023 15:55
display: flex;
justify-content: start;
flex-wrap: wrap;
gap: 1em 3em;

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Refactored the link list in the section under the TeachAI announcement to work w/ rtl languages (thanks @megcrenshaw!)
Screenshot 2023-04-25 at 9 06 30 AM

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hurray! Looks great! This has been a huge learning curve for me too 📖 🥳

%p{style: "color: white"}=hoc_s(:ai_hero_desc)
- if !!DCDO.get('teach-ai-launch-2023', false)
%a.link-button{href: "#teach-ai-callout"}
%i{class: "#{icon_megaphone}", style: "margin-right: 6px"}

@kelbyhawn kelbyhawn Apr 25, 2023

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I need to make a button class that will work with icons for rtl languages bit need to think about it more and will do that in another task when there's more time (ACQ-551)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Sounds good!

%p.no-margin-bottom=hoc_s(:ai_intro_list_desc_03)
.clear

%a{id: "teach-ai-callout"}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I'm confused by this –– does this produce a whole section that's a link? Should this be inside the DCDO conditional?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It's a tag so the button in the hero banner knows where to jump to when clicked…but I just looked up this article and you can add the id tag to elements on the page (like a section) that essentially does the same thing w/o needing a hanging a tag — I'll move all of the other id anchors to their respective sections and get rid of the hanging as!

So much learning this week, I love it! 🤓 🙌

i {
font-size: 1.25em;
margin: 0 4px 0 0;
margin: 3px 0 0;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Wow I didn't know what 3 arguments did until today! Cool.

@megcrenshaw megcrenshaw left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I have one question, but not blocking! Looks great!

@kelbyhawn
kelbyhawn merged commit ba21858 into staging Apr 25, 2023
@kelbyhawn
kelbyhawn deleted the ai-page-teachai-blog-blurb branch April 25, 2023 20:02
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