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

Move links for github/videos to bottom of cell #14

Closed
wants to merge 4 commits into from

Conversation

richa2710
Copy link
Contributor

@richa2710 richa2710 commented Dec 4, 2023

Moved the links for Github and Videos at the bottom of each cell.

Closes #2

@klieret klieret changed the title Training richa Move links for github/videos to bottom of cell Dec 4, 2023
@@ -79,8 +80,8 @@ code {

.tutCard .videos {
position: absolute;
top: 2.5rem;
right: 0.8rem;
bottom: 0.3rem;
Copy link
Member

Choose a reason for hiding this comment

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

With this specification I wonder: If there's a lot of text in the box, will this end up on top of the text? Can you try this by modifying the data?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Right! And the size of the box is also fixed. So it doesn't scale with the text
Screenshot 2023-12-04 at 10 05 41 AM

left: 0;
}

.tutCard .videos {
Copy link
Member

Choose a reason for hiding this comment

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

duplicate?

@klieret klieret self-requested a review December 18, 2023 21:21
@klieret
Copy link
Member

klieret commented Dec 27, 2023

I think the easy way to do this is what I had started in #5, which also solves all issues related to text wrapping etc.

I'm going to close this, it's better to start fresh. Sorry for that!

@klieret klieret closed this Dec 27, 2023
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.

Make links to website/repository/videos more obvious
2 participants