Description
Design and QA review of the new Angular dotAI portlet (#37417) turned up seven follow-ups — five UI/copy corrections and two bugs in the Embeddings tab. Grouping them into one issue since they all land in the same portlet and are best fixed and QA'd together.
- Empty states don't match the platform. The portlet rolled its own empty state instead of the
dot-empty-container every other portlet uses. Replace it everywhere.
Fields and Velocity template need real descriptions. The Velocity template has no hint and no placeholder at all, and the Fields hint is wrong — it says "leave empty to embed the whole contentlet" when dotCMS actually picks a subset of fields. The two are also alternatives, not complements: a Velocity template overrides Fields entirely. None of that is visible in the dialog. Needs a hint on both plus an example placeholder on the template.
- Remove "View provider config". The button and its JSON dialog in Config Values aren't needed.
- "Rebuild DB" is red. Should be a regular outlined button — the destructive step is already the confirm dialog.
- The Config Values host line is unclear. It reads
demo.dotcms.com (falls back to system host), which doesn't say what it means and isn't translatable.
- Build warnings appear behind the modal. Submitting the Build an index dialog closes it, then shows the outcome — including Lucene query errors — in the tab behind it. The user loses what they typed and has to start over. Keep the modal open and show it there.
- A new index doesn't appear in the table. After building, the row only shows up after a manual page reload.
Acceptance Criteria
Additional Context
Follow-up to #37417 (Dojo to Angular: dotAI Portlet), implemented in #37423. Feedback from Jorge Conde (9/11) and melissa.rojas (9/10).
Items 6 and 7 are defects; 1–5 are UI/copy. Items 2 and 5 touch the backend; the rest are frontend-only.
Description
Design and QA review of the new Angular dotAI portlet (#37417) turned up seven follow-ups — five UI/copy corrections and two bugs in the Embeddings tab. Grouping them into one issue since they all land in the same portlet and are best fixed and QA'd together.
dot-empty-containerevery other portlet uses. Replace it everywhere.FieldsandVelocity templateneed real descriptions. The Velocity template has no hint and no placeholder at all, and the Fields hint is wrong — it says "leave empty to embed the whole contentlet" when dotCMS actually picks a subset of fields. The two are also alternatives, not complements: a Velocity template overrides Fields entirely. None of that is visible in the dialog. Needs a hint on both plus an example placeholder on the template.demo.dotcms.com (falls back to system host), which doesn't say what it means and isn't translatable.Acceptance Criteria
dot-empty-containerfrom@dotcms/uiand matches the empty states in other portletsVelocity templatehas a hint and an example placeholder; theFieldshint is corrected, and it's clear a template overrides FieldsView provider configbutton and dialog are goneRebuild DBrenders as a regular outlined buttonBuildingright after submit and flips toReadyon its own, with no page reloadAdditional Context
Follow-up to #37417 (Dojo to Angular: dotAI Portlet), implemented in #37423. Feedback from Jorge Conde (9/11) and melissa.rojas (9/10).
Items 6 and 7 are defects; 1–5 are UI/copy. Items 2 and 5 touch the backend; the rest are frontend-only.