Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Category nesting #1

Closed
yellowled opened this Issue Sep 8, 2011 · 5 comments

Comments

Projects
None yet
2 participants
Owner

yellowled commented Sep 8, 2011

The categories plugin should get a proper markup to display nested categories, i.e. nested unordered lists. This saves additional classes or inline styles for indenting nested classes because nested ul and li elements can be addressed via the CSS cascade.

Contributor

levito commented Sep 13, 2011

Would be awesome for threaded comments, too! :)

Owner

yellowled commented Sep 13, 2011

See, I'm not so sure about that. The categories list could be considered a navigation list (but not a nav element), but the comments most definitely are not a navigation element. Is a ul (or even ol) really the proper HTML element for comments?

Contributor

levito commented Sep 13, 2011

I think of a list of comments. Semantically, nesting lists for threading makes perfect sense to me. It also opens up new possibilities for styling nested comments. See these examples:
http://bavotasan.com/wp-content/uploads/2009/03/threaded.png
http://images.patterntap.com/9/6/9630087694cffa810b1f03.png

Owner

yellowled commented Sep 17, 2011

I have given this some thought over the past couple of days. I guess my main concern remains: Is a list with just one item (which will happen in nested uls for threaded comments) still properly marked up as a list or is this using HTML as presentational markup?

Owner

yellowled commented Sep 30, 2011

Update: Garvin basically ruled out comment nesting since it's really expensive performance-wise. I'm actually quite okay with the current comment nesting, but I'd still like to have it for categories, which apparently is possible.

@yellowled yellowled closed this Apr 4, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment