-
-
Notifications
You must be signed in to change notification settings - Fork 456
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
Patch3 #1051
Patch3 #1051
Conversation
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 is recommended to read the contributing guidelines.
Follow the correct directory structure of the repository. All files like css
, js
, video
, images
.. are in assets directory and html
files are in Public directory
Could you please update the file paths in the code to reflect the recent changes? It's important to ensure that the paths are correctly pointing to the updated files. If you need any assistance with this, feel free to let me know.
Additionally, it would be great if you could apply a white background color to the navigation section. This is an optional suggestion, but I highly recommend implementing it as it can improve the overall aesthetics and readability of the navigation area.
Please don't hesitate to reach out if you have any further questions or need any further assistance. I'm here to help!
Let me know if there are any specific points you'd like to discuss or clarify.
</svg> | ||
</a> | ||
<nav class="center desktop"> | ||
<a href="im">Model S</a> |
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.
<a href="im">Model S</a> | |
<a href="">Model S</a> |
To ensure a consistent user experience, it is important to avoid confusing users when they click on a link and encounter an error. Therefore, it would be beneficial to make the linking consistent.
Additionally, it would greatly enhance user convenience if you could implement internal linking within the sections of the website from the navbar. This functionality should be available on both mobile and desktop versions.
Thank you @vijaynaidu16 for taking out your valuable time in order to contribute to our project. Looking forward for more such amazing contributions :) |
π οΈ Fixes Issue (Number)
Closes #issue_number
π¨βπ» Changes proposed
I made a Tesla UI clone form its website using HTML,CSS,JavaScript and make it responsive
β Check List (Check all the applicable boxes)
π Note to reviewers
π· Screenshots