Add User Feedback buttons

[christian-byrne]

This PR adds a Feedback button to the Help topmenu submenu, which opens the feedback dialog, allowing users to send any feedback they have.

On desktop, a feedback button already existed which linked to Comfy Forum - Desktop v1 Feedback. This PR replaces it. It can be re-added as a separate button later.

┆Issue is synchronized with this Notion page by Unito

[huchenlei]

I feel like the feedback button as a badge in the about panel might not be the best design. Maybe we would want it to be in a separate section?

For this PR let's just drop the badge related change and only trigger feedback dialog with top dropdown menu, and leave the redesign of about panel to a separate PR.

[huchenlei]

We do have extensions like ComfyUI-Manager adding custom badge to the about panel right now. Changing this interface would break their implementation.

[huchenlei]

Changes:

• Replace non-existing menuLabels.Feedback with g.feedback.
• Add min-w to the dialog so that the dialog holds proper width even in other locales.
  
  
• Validate email format

[christian-byrne]

Changes:

• Replace non-existing menuLabels.Feedback with g.feedback.
• Add min-w to the dialog so that the dialog holds proper width even in other locales.

Thank you!

[huchenlei]

@christian-byrne I think the rating field can be further discussed separately. Currently we don't have feature category. The overall rating to the whole ComfyUI app might not be that useful. If later we can have feature tags, it might make more sense to add the rating.

Ref:
https://issues.chromium.org/issues/390936402

[christian-byrne]

SGTM. In that case, I can actually simplify this a lot and just render the feedback dialog using the method in this #2284

[huchenlei]

SGTM. In that case, I can actually simplify this a lot and just render the feedback dialog using the method in this #2284

I think the overall thinking process is that rating only gives value when the rating of one thing is compared with the rating of another thing. If there is only one category, then the value actually doesn't give much information to us.

[huchenlei]

UI after change:

[christian-byrne]

Okay, but this no longer requires its own component. It can use the same function to render dialog as the one proposed in #2284. So let me simplify the two PR into one.

[huchenlei]

Okay, but this no longer requires its own component. It can use the same function to render dialog as the one proposed in #2284. So let me simplify the two PR into one.

Let's merge this PR and refactor it to support #2284.

[christian-byrne]

ComfyUI_frontend/src/components/dialog/content/FeedbackDialogContent.vue

Lines 23 to 27 in 0142edb

	<style scoped>
	:deep(.p-panel) {
	@apply border-none;
	}
	</style>

Generally better to use passthrough:

    <ReportIssuePanel
      error-type="Feedback"
      :title="$t('issueReport.feedbackTitle')"
      :default-fields="['SystemStats', 'Settings']"
      :pt="{
        root: {
          class: 'border-none'
        }
      }"