Skip to content

Enhance default message logic and setting page layout#14

Open
jimsihk wants to merge 1 commit into
e-abc:masterfrom
jimsihk:feat/settings-form-layout
Open

Enhance default message logic and setting page layout#14
jimsihk wants to merge 1 commit into
e-abc:masterfrom
jimsihk:feat/settings-form-layout

Conversation

@jimsihk

@jimsihk jimsihk commented Feb 11, 2024

Copy link
Copy Markdown

I would like to propose below changes to the setting pages layout:

  1. Add section headers to the setting page to make it clearer on different type of the notifications
  2. Display the default values in the settings page
  3. Add a help button in the course setting page for the status dropdown to describe the usage in more details

Since the default message values are using different placeholders that site admin could use, I suggest to also revise the logic to align the placeholder values for better maintenance and display.

Note: Sorry that I am not familiar on Spanish so the corresponding translations are not added to the ES translation file...

@jimsihk jimsihk force-pushed the feat/settings-form-layout branch from 37a835a to b82ff59 Compare February 11, 2024 22:04
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.

1 participant