ConfirmDialog: Unstyled mode is not implemented

[zawasp]

Describe the bug

Comparing the ConfirmPopup code with ConfirmDialog, the code which takes into account unstyled mode is missing completely.

10.5.0

React version

18.x

Language

TypeScript

Build / Runtime

Next.js

Browser(s)

No response

Steps to reproduce the behavior

Expected behavior

Unstyled mode can be applied.

[zawasp]

Any updates on this one?

[melloware]

Any investigation or PR is welcome!

[zawasp]

@melloware it seems ConfirmDialog doesn't use unstyled mode at all. Just comparing ConfirmPopup and ConfirmDialog code, there's nothing related to it.

[melloware]

Agreed.

[zawasp]

I'll edit the title and description then, to reflect the new finding.

[melloware]

@kl-nevermore i thought you fixed ConfirmDialog in this ticket: #5656

[zawasp]

The equivalent default Tailwind style confirmdialog is for sure missing in https://github.com/primefaces/primereact/blob/master/components/lib/passthrough/tailwind/index.js (like we have for https://github.com/primefaces/primereact/blob/master/components/lib/passthrough/tailwind/index.js#L373),

as well as these lines:
https://github.com/primefaces/primereact/blob/master/components/lib/confirmpopup/ConfirmPopup.js#L42-L44

[melloware]

@zawasp want to submit a PR?

[zawasp]

I would if I could start the dev build of primereact so I can debug properly.

[zawasp]

@

[kl-nevermore]

@kl-nevermore i thought you fixed ConfirmDialog in this ticket: #5656

I forgot to check if unstyled mode is implemented, I will recheck the previous changes are unnecessary

[kl-nevermore]

checked
These two are different issues

[melloware]

Yep we have updated confirm dialog in the latest version and added the new styles