Skip to content

Update Properties#16

Open
rgant wants to merge 1 commit intobrandon-rhodes:masterfrom
rgant:rgant-patch-1
Open

Update Properties#16
rgant wants to merge 1 commit intobrandon-rhodes:masterfrom
rgant:rgant-patch-1

Conversation

@rgant
Copy link

@rgant rgant commented Aug 29, 2022

From list on MDN

From list on MDN
@brandon-rhodes
Copy link
Owner

Wow! That's a lot of properties. I still do CSS occasionally but had no idea there were now that many. The next time I'm in CSS mode, I'll try learning about a few of these new concepts like "gap" to see if their placement in the file makes sense to me, and either merge or make further comments here. Thanks!

@rgant
Copy link
Author

rgant commented Sep 11, 2022

No offense, I kinda thought this repo was abandoned. So I failed to put a lot of thought into making it nice for you to review.

I would be happy to do some cleanup on this with commenting and more thought to the order of things.

Personally I just alphabetize my css properties, but the front-end developers I work with tell me concentric makes more sense to them. My goal for updating this list is so my CSS formatter could arrange the properties automatically how they want it. I'm not really sure I made the order make sense all the time. But if you are actually going to review I can re-read your blog and see if I have done it your way.

Let me know when you think you will have time and I can add this to one of my sprints for around then.

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