@decidim/product all your feedback from #10468 has been implemented or delegated to another issue. Please make another review and feel free to open new issues, this PR is frozen except for critical bugs

@alecslupu when that happens, you need to restart bin/webpack-dev-server

@alecslupu when that happens, you need to restart bin/webpack-dev-server

@Crashillo I am not using de the webpack dev server ... I just checked out the branch, removed the asset folder in my public, ran decidim:upgrade , restared the webserver ...

In any case, comments is a branch who introduces a new pack (decidim_comments), so every single time you switch among branches, you'll have to restart the assets compiler, otherwise the manifest file won't have such entries.

When we develop, we run two tabs up: bin/rails server, and bin/webpack-dev-server (hot-reload) or bin/webpack(only compilation). Switching branches throws that error you mentioned, so restart the frontend server. Or run it once at least.

running webpack-dev-server ... i get this error:

ERROR in ./decidim/decidim-comments/app/packs/stylesheets/comments.scss (./decidim/decidim-comments/app/packs/stylesheets/comments.scss.webpack[javascript/auto]!=!./node_modules/css-loader/dist/cjs.js??ruleSet[1].rules[4].use[1]!./node_modules/postcss-loader/dist/cjs.js??ruleSet[1].rules[4].use[2]!./packages/webpacker/src/loaders/decidim-sass-loader.js!./decidim/decidim-comments/app/packs/stylesheets/comments.scss)
Module build failed (from ./node_modules/postcss-loader/dist/cjs.js):
SyntaxError

(64:4) <css input> The `first:[&_a[href^="/profile"]>span]:z-10` class does not exist. If `first:[&_a[href^="/profile"]>span]:z-10` is a custom class, make sure it is defined within a `@layer` directive.

  62 | 
  63 |   &__header{
> 64 |     @apply flex items-center gap-x-2 mb-2 -ml-[calc(1.5rem+.75rem)] relative first:[&_a[href^="/profile"]>span]:z-10;
     |    ^
  65 | 
  66 |     &--edited{

ERROR in ./decidim/decidim-comments/app/packs/stylesheets/comments.scss
Module build failed (from ./node_modules/mini-css-extract-plugin/dist/loader.js):
HookWebpackError: Module build failed (from ./node_modules/postcss-loader/dist/cjs.js):
SyntaxError

I will reinstall my webpacker, to make sure is not that the root cause of my errors.

If you re-create the development_app it works, maybe you should try.

Anyway we have an issue with the commit 9f4ccd7 which broke many tests, we are fixing it right now.

If you re-create the development_app it works, maybe you should try.

Anyway we have an issue with the commit 9f4ccd7 which broke many tests, we are fixing it right now.

I got it working ... was a Mishap on my end ( I use multiple containers and something got mangled by a config) ...
I was just getting the same error as you did ... ActionView::Template::Error (undefined method flag_modal'` .. waiting for fix. :)

@alecslupu ready to review again

When a comment is being moderated the view is rendered as follows:

When a comment is being moderated the view is rendered as follows:

That's something solved in staging, don't have to worry:

https://decidim-redesign.populate.tools/processes/smile-cutting/f/19/posts/10

That's something solved

@ferblape this is NOT solved ...

Check here : https://decidim-redesign.populate.tools/processes/smile-cutting/f/19/posts/10

Ready to re-review @alecslupu, ping me if you want clarification about the commented specs you want to remove

@alecslupu could you review this today? we need some stuff from this PR in the filters branch

Team, thanks for the work already done. There are some improvements that could be done:

1. The emoji box is not aligned
2. There is a refresh happening in the comments, as per below video, that is not happening all the time.