Skip to content

Add space between mandatory field name and asterisk - #3250

Merged
alicia-koh merged 10 commits into
developfrom
alicia-koh/wizard-config-required-fields
Sep 3, 2026
Merged

Add space between mandatory field name and asterisk#3250
alicia-koh merged 10 commits into
developfrom
alicia-koh/wizard-config-required-fields

Conversation

@alicia-koh

@alicia-koh alicia-koh commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator

Merge Checklist

Screenshot 2026-09-02 at 2 57 00 PM

Please cross check this list if additions / modifications needs to be done on top of your core changes and tick them off. Reviewer can as well glance through and help the developer if something is missed out.

  • Automated Tests (Jasmine integration tests, Unit tests, and/or Performance tests)
  • Updated Manual tests / Demo Config
  • Documentation (Application guide, Admin guide, Markdown, Readme and/or Wiki)
  • Verified that local development environment is working with latest changes (integrated with latest develop branch)
  • following best practices in code review doc

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds consistent visual spacing between mandatory field labels and their asterisks.

Changes:

  • Increases group-required asterisk spacing to 4px.
  • Adds matching 4px spacing for field-required asterisks.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@alicia-koh
alicia-koh enabled auto-merge September 3, 2026 07:42
@alicia-koh
alicia-koh added this pull request to the merge queue Sep 3, 2026
Merged via the queue into develop with commit ab79686 Sep 3, 2026
70 of 72 checks passed
@alicia-koh
alicia-koh deleted the alicia-koh/wizard-config-required-fields branch September 3, 2026 10:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants