[PR #204] [MERGED] Added table of contents and reordered email settings text to make it more clear #301

New issue

Closed

opened 2026-02-26 10:30:42 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 10:30:42 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/opensolutions/ViMbAdmin/pull/204
Author: @diogocsc
Created: 9/7/2016
Status: ✅ Merged
Merged: 9/8/2016
Merged by: @barryo

Base: master ← Head: master

📝 Commits (4)

3d6623b Text summary and reorder
656b532 Revert "Text summary and reorder"
16e4b8d Table of contents and reorder
457ff1a Merge pull request #1 from diogocsc/Rearrange_EmailSettings

📊 Changes

1 file changed (+5 additions, -2 deletions)

View changed files

📝 application/views/mailbox/email/settings.phtml (+5 -2)

📄 Description

Given the extent of the settings email, users may need to scroll, and so miss part of the email text. Adding a table of contents helps reader to be aware what the email content is.

Also, the "check settings below" line is more effective after the password change message, as the reader will be able to understand what the sentence refers too (which is not the password change as it may be lead to believe with current text)

I agree to the Contributor License Agreement

@pierreozoux check this out.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/opensolutions/ViMbAdmin/pull/204 **Author:** [@diogocsc](https://github.com/diogocsc) **Created:** 9/7/2016 **Status:** ✅ Merged **Merged:** 9/8/2016 **Merged by:** [@barryo](https://github.com/barryo) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (4) - [`3d6623b`](https://github.com/opensolutions/ViMbAdmin/commit/3d6623b3f70276b4e2d3edae856212f24eefb2f8) Text summary and reorder - [`656b532`](https://github.com/opensolutions/ViMbAdmin/commit/656b532e6313421f021d6ff5bb298637d20aa420) Revert "Text summary and reorder" - [`16e4b8d`](https://github.com/opensolutions/ViMbAdmin/commit/16e4b8dcfeabf4399005fe8bd79a760dc3b7030e) Table of contents and reorder - [`457ff1a`](https://github.com/opensolutions/ViMbAdmin/commit/457ff1a1ba158cb5d102078160325f245f85799a) Merge pull request #1 from diogocsc/Rearrange_EmailSettings ### 📊 Changes **1 file changed** (+5 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `application/views/mailbox/email/settings.phtml` (+5 -2) </details> ### 📄 Description Given the extent of the settings email, users may need to scroll, and so miss part of the email text. Adding a table of contents helps reader to be aware what the email content is. Also, the "check settings below" line is more effective after the password change message, as the reader will be able to understand what the sentence refers too (which is not the password change as it may be lead to believe with current text) I agree to the [Contributor License Agreement](https://github.com/opensolutions/ViMbAdmin/wiki/Contributor-License-Agreement) @pierreozoux check this out. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>