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

making images responsive #825

Conversation

DanDaManTran
Copy link
Contributor

No description provided.

@DanDaManTran DanDaManTran merged commit f2da03f into main Jun 11, 2024
1 check passed
@DanDaManTran DanDaManTran deleted the LD-204-fix-bug-in-academy-where-images-can-overflow-the-main-content-in-academy-they-should-be-responsive branch June 11, 2024 16:05
@@ -81,7 +81,8 @@ Create routes for the `HomeComponent` and `RestaurantComponent`. When the route
Notice that you will be able to click the **Choose a Restaurant** button
after implementing the solution:

![Choose a restaurant routing](../static/img/angular/4-adding-routing/after.gif 'Choose a restaurant routing')
<img alt="Choose a restaurant routing." src="../static/img/angular/4-adding-routing/after.gif"
style="border: solid 1px black;" width="90%"/>
Copy link
Member

Choose a reason for hiding this comment

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

Why was this set to 90% and not 100%?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

hmm I thought I set the width with similar widths in Angular. I guess not.

Copy link
Member

Choose a reason for hiding this comment

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

I added requirements to the issue: https://bitovi.atlassian.net/browse/LD-204

Please let me know if you have any questions, I’m happy to talk through it more.

@@ -18,7 +18,7 @@ In this section, we will:

We want to have a component to display individual restaurants details, and want the path to be nested under the restaurants path.

<img alt="Screenshot of the Place My Order app with a nested route, displaying the “/restaurants/lettuce-shack” URL in the browser’s address bar." src="../static/img/react-vite/10-nested-routes/01-solution.png" style="max-width: 735px;"/>
<img alt="Screenshot of the Place My Order app with a nested route, displaying the “/restaurants/lettuce-shack” URL in the browser’s address bar." src="../static/img/react-vite/10-nested-routes/01-solution.png" style="max-width: 640px;"/>
Copy link
Member

Choose a reason for hiding this comment

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

What’s this value based on? I just checked the image and it should be a max of 735px wide.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

735px image was overlapping with the side bar. 640px was used in /src/react-vite/03-intro-to-jsx/intro-to-jsx.md so I went with that.

Copy link
Member

Choose a reason for hiding this comment

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

Let’s move these values out of the style attribute and specify the height and width attributes on the image: https://bitovi.atlassian.net/browse/LD-204

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