Skip to content

Branch1 #685

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

Open
wants to merge 37 commits into
base: main
Choose a base branch
from
Open

Branch1 #685

wants to merge 37 commits into from

Conversation

Prateek14b
Copy link

@Prateek14b Prateek14b commented May 31, 2025

Because

This PR

Issue

Closes #XXXXX

Additional Information

Pull Request Requirements

  • I have thoroughly read and understand The Odin Project Contributing Guide
  • The title of this PR follows the location of change: brief description of change format, e.g. 01-flex-center: Update self check
  • The Because section summarizes the reason for this PR
  • The This PR section has a bullet point list describing the changes in this PR
  • If this PR addresses an open issue, it is linked in the Issue section
  • If applicable, I have ensured that the TOP solution files match the Desired Outcome image

Prateek14b and others added 30 commits February 2, 2025 19:53
…value for padding refers to up/down padding, second value refers to left/right padding.)
…ligned it to the right. THus, fourth bullet point is addressed.
…or div element and made required style modifications.
…ecified color to a declaration that specified background color (like the requirements asked)
…s, and when grouping selectors. Here, chaining selectors are the way to go.
…to give confirm button precedence over button. Then, since .button selector's properties suited the cancel button's properties, didn't bother to create a chaining selector .button.cancel.
…g div.text selector, for applying special styles to child elements.
…nt on CSS, for easy debugging. Second, ALWAYS set box-sizing to border-box for ALL elements, using the * selector.
…i have no idea what how but it just works. move on
Prateek14b and others added 7 commits May 30, 2025 22:29
… up with an exact match to the desired picture.
…from the solution.css and solution.html that i need to be more minimalistic with my approach, not relying too much on flex containers and using too many wrappers that the htmls and css becomes harder to understand and, thus, scale. BE MINIMALIST.
…property to the image in between, to allow it to consume any remaining space on the webpage and grow -- or shrink, whatever context.
…ue with css, to get the desired layout, in the morn.
…sing the flex property. set flex-shrink to 0 for sections of the webpage you want the same size, regardless of browser window size, and set flex:1 for sections that you want to let grow when expanded.
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.

1 participant