-
Notifications
You must be signed in to change notification settings - Fork 34
Improve methods types styles #534
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
base: main
Are you sure you want to change the base?
Conversation
|
Thank you for working on the PR but unfortunately i don't agree with this change. The method title/call sequence is one of the first things users look at & want to see when arriving on the page, we want these headers to be super easy to distinguish from everything else. Making the headers smaller makes it blend in more easily with method descriptions and harder to distinguish for users. |
|
Small note: I've checked default sizes for HTML headers and compared to Tailwind, there are they:
So… they're different, but look approximately equal. Also we have |
|
@colby-swandale did you see my comment on the "split class and instance methods" that this PR is trying to resolve? I haven't loaded this PR up, so I can't comment on that, but something really should change compared to master. |
|
Sorry @natematykiewicz, I did see the comment but didn't get around to replying. I agree that something needs to change between the section titles & method titles, but i am hesitant to change the method title styles as they are - in my mind - one of the most important things to the user on the page. The solution in my view is to do something similar that I did in #382 (comment) and have the section headers styled differently & spaced far between method headers. |
|
Sorry I should also say that I am open to other ideas, but I am just hesitant to change the current style of the method headers unless there is a clear benefit for the user. |
|
Can someone post a screenshot of what this PR looks like before and after? |


Resolve #382 (comment)