Add dropdown demo #1
Conversation
|
@jamesnw I have some style cleanup edits I thought would be easier to do locally. Could you grant me access here? |
|
@dvdherron Whoops, sorry- you should have access now! |
|
@stacyk This is ready for style review whenever you're ready |
src/dropdown/style.css
Outdated
| ul { | ||
| position-area: span-block-end inline-end; | ||
| position-try: block-end span-inline-end; | ||
| } |
There was a problem hiding this comment.
@dvdherron Is this the clearest way of selecting 2nd level menus? I'm wondering if the selector should be .menu here as well?
There was a problem hiding this comment.
Oh, I thought I saw the .menu selector on the submenus- but I didn't. Should there be?
There was a problem hiding this comment.
There was a problem hiding this comment.
Yup, that's great! Thanks! I think that's clearer than just basing it on semantics and hierarchy.
✅ Deploy Preview for oddbaseline ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
2 participants
Run
npx http-server .Open http://localhost:8080/src/dropdown