Skip to content

Hacktoberfest -- update index.html <html> and <head>#18

Open
famanakis wants to merge 2 commits into
yash19sinha:mainfrom
famanakis:code-enhancement
Open

Hacktoberfest -- update index.html <html> and <head>#18
famanakis wants to merge 2 commits into
yash19sinha:mainfrom
famanakis:code-enhancement

Conversation

@famanakis

Copy link
Copy Markdown

Added language to tag
Added viewport meta data to

Added language to <html> tag
Added viewport meta data to <head>
@vercel

vercel Bot commented Oct 7, 2023

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
coffee-bean ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 7, 2023 4:00pm

@yash19sinha

Copy link
Copy Markdown
Owner

It's very basic thing, try to make some more valuable changes

HTML - removed inline styles for img in the menu.  Added alt to each img tag, for accessibility purposes.
CSS - organized, grouped, categorized CSS for readability.  Added box-sizing to all.  Added styling from HTML to CSS for card images in Menu.  Added flex property with justify content set to center for .card-body so images are centered in the menu.
@famanakis

Copy link
Copy Markdown
Author

I did some more work on this project. Added alt to the img tags. Moved inline styles for img in the menu section to the CSS. Centered the images in the menu. Organized the CSS and gave titles to some sections for readability and ease of use in the code base.

@maitri-vv

Copy link
Copy Markdown
Collaborator

@famanakis Thanks for adding more changes, but you need to resolve conflicts first now.

@yash19sinha

yash19sinha commented Oct 9, 2023

Copy link
Copy Markdown
Owner

I think in you should first fetch all the changes done by your fellow contributors and then can do your commits
Use commands like git pull every time before you start doing some changes and make sure that your branch is updated every time @famanakis

@famanakis

famanakis commented Oct 10, 2023 via email

Copy link
Copy Markdown
Author

@yash19sinha

Copy link
Copy Markdown
Owner

@famanakis in this kind of situation you need to accept the incoming changes and then should figure out how to merge the code in such a way that the previous changes and your new changes go properly without overlapping

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