fiatjaf / nocomment

See a demo at https://fiatjaf.com/nostr.html
https://nocomment.pages.dev/embed.js
136 stars 17 forks source link

Please provide an easy to include widget #1

Closed Giszmo closed 11 months ago

Giszmo commented 2 years ago

This issue is a reminder for what is clearly missing. Please refine ...

Right now, the <aside> tag just gets filled with divs with inline style, making it hard to style but it's not far from being cool and flexible.

johnsBeharry commented 2 years ago

Is your use case using a javascript framework or you prefer it to be vanilla js? We have a react component that we wanna publish and make customisable and importable. It's not currently hooked up to Nostr though.

https://makers-bolt-fun-stories.netlify.app/?path=/story/posts-components-comments-commentssection--default

Would this be useful for you?

Giszmo commented 2 years ago

@johnsBeharry at this point I'm very much invested in nostr and for my use case the decentralized nature of where the comments are stored is essential, too, so if your "react component" uses your company servers or something like that, it wouldn't be of interest for me even if there were no nostr.

johnsBeharry commented 2 years ago

You're preaching to the choir man.

verityj commented 1 year ago
  • [ ] Put style into a .css

Not sure how to do that, but for me it was helpful to add additional variables. Those are easy to set in your .css.

.nocomment {
  --nc-background:
  --nc-container-font-family:
  --nc-container-font-size:
  --nc-comment-author-font-size:
  --nc-comment-author-font-family: 
  --nc-comment-author-font-color: 
  --nc-comment-date-color: 
  --nc-comment-date-font-family: 
  --nc-comment-date-font-size: 
  --nc-link-text-decor: none;
  --nc-link-text-decor-hover: none;
  --nc-textarea-font-family: 
  --nc-textarea-font-size: 
  --nc-text-background: 
  --nc-text-color: 
  --nc-text-color-dark: 
  --nc-primary-color: 
  --nc-primary-contrast: 
}

https://github.com/fiatjaf/nocomment/pull/21

Giszmo commented 11 months ago

I think this issue is kind of resolved.