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
notes drawer styles #308
notes drawer styles #308
Conversation
✅ Deploy Preview for bldrs-share ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
@pablo-mayrgundter please take a look |
Looking better! There's a problem with paging.. load KNIK and then page issues.. you can get to 3/4 but not to 4/4. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/docs directory shouldn't be part of this or future PRs. Did you sync to head to create this branch? Might want to restart it..
This looks like it's pulling in Markus's proposal to remove metadata. Could you list in the description which issues you're addressing with this PR? I don't agree with that change.. I think it's right for smth like Portfolio mode when we have that, but in general I think it helps orient the user that these are indeed comments to see the usual metadata. |
…into Notes_styles
@pablo-mayrgundter PTAL |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
One change for unit tests.. looks like you removed the assert for username in previous change, but should be back now. Added comment below...
Address the following issues:
#304
part of this issue
#306
Adjusted: