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

website changes #340

Merged
merged 3 commits into from
Apr 8, 2023
Merged

website changes #340

merged 3 commits into from
Apr 8, 2023

Conversation

ageddesi
Copy link
Owner

@ageddesi ageddesi commented Apr 8, 2023

Pull request template

IMPORTANT: Please review the CONTRIBUTING.md file for detailed contributing guidelines.

Describe your changes

Link to issue this resolves

Screenshot of changes(if relevant)

Aleksander-Dahle and others added 3 commits April 6, 2023 23:35
The first bug I fixed was related to the header of the web page. There was a div element with the class ml-10 space-x-8 that was causing the HTML to break when the page was viewed on smaller screens.

The second bug I fixed was related to the color of a logo on the web page. The original logo was black, now its changed to white.
change the inline styling to tailwind
@ageddesi ageddesi merged commit b706f9e into main Apr 8, 2023
@what-the-diff
Copy link

what-the-diff bot commented Apr 8, 2023

PR Summary

  • Dependency update
    Added yarn to improve package management.
  • UI improvement for svg icon
    Adjusted the icon color for better visibility in Weathers category.
  • Fixed header misalignment
    Removed unnecessary CSS class to ensure proper alignment on mobile devices.

@ghost
Copy link

ghost commented Apr 8, 2023

👇 Click on the image for a new way to code review

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

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.

2 participants