Skip to content

Conversation

@SanidhyaSharma28
Copy link
Contributor

can you check if its fine?

@vercel
Copy link

vercel bot commented Oct 23, 2023

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

Name Status Preview Comments Updated (UTC)
acm-fun ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 23, 2023 7:43pm

Copy link
Contributor

@Ishwarendra Ishwarendra left a comment

Choose a reason for hiding this comment

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

Also complete black doesn;t look too good. Try to keep light background for now to match other games and activities.

Comment on lines 4 to 5
width: 100vw;
height: 100vh;
Copy link
Contributor

Choose a reason for hiding this comment

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

Don't hardcode these values.

image

There should not be any horizontal scrollbars.

color: white;
}

body {
Copy link
Contributor

Choose a reason for hiding this comment

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

dont' add css to body, h1, div p and similar tags that can affect other components as well.

color: white;
}

.board {
Copy link
Contributor

Choose a reason for hiding this comment

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

Name css class like wordleBoard instead of board.

@SanidhyaSharma28
Copy link
Contributor Author

@Ishwarendra can you check now???

Copy link
Contributor

@Ishwarendra Ishwarendra left a comment

Choose a reason for hiding this comment

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

Good work.

@Ishwarendra Ishwarendra merged commit dabbe2f into PEC-CSS:master Oct 23, 2023
@Ishwarendra Ishwarendra linked an issue Oct 23, 2023 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Wordle

2 participants