[PR #669] [MERGED] Added a warn when user closes tab or browser with unsaved changes to … #971

Closed
opened 2026-02-25 21:36:49 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/cypht-org/cypht/pull/669
Author: @josaphatim
Created: 2/14/2023
Status: Merged
Merged: 12/22/2023
Merged by: @kroky

Base: masterHead: warn-for-unsaved-changes


📝 Commits (3)

  • 25f66a5 Added a warn when user closes tab or browser with unsaved changes to settings
  • 6811cfc Added are you sure on forms if user setting is activated
  • f5c2b6b Fixed tests

📊 Changes

24 files changed (+283 additions, -22 deletions)

View changed files

📝 language/az.php (+2 -2)
📝 language/de.php (+2 -1)
📝 language/en.php (+2 -1)
📝 language/es.php (+2 -1)
📝 language/et.php (+2 -1)
📝 language/fa.php (+2 -1)
📝 language/fr.php (+2 -1)
📝 language/hu.php (+2 -1)
📝 language/id.php (+2 -1)
📝 language/it.php (+2 -1)
📝 language/ja.php (+2 -1)
📝 language/nl.php (+2 -1)
📝 language/pt-BR.php (+2 -1)
📝 language/ro.php (+2 -1)
📝 language/ru.php (+2 -1)
📝 language/zh-Hans.php (+1 -0)
📝 modules/core/handler_modules.php (+17 -0)
📝 modules/core/output_modules.php (+23 -0)
📝 modules/core/setup.php (+4 -1)
📝 modules/core/site.js (+5 -1)

...and 4 more files

📄 Description

If someone closes browser, or tab and has unsaved changes to settings this attempts to warn him.

Fixes: - https://github.com/jasonmunro/cypht/issues/523


🔄 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/cypht-org/cypht/pull/669 **Author:** [@josaphatim](https://github.com/josaphatim) **Created:** 2/14/2023 **Status:** ✅ Merged **Merged:** 12/22/2023 **Merged by:** [@kroky](https://github.com/kroky) **Base:** `master` ← **Head:** `warn-for-unsaved-changes` --- ### 📝 Commits (3) - [`25f66a5`](https://github.com/cypht-org/cypht/commit/25f66a5b5d55b0cf86368538662d54a90a8db8e7) Added a warn when user closes tab or browser with unsaved changes to settings - [`6811cfc`](https://github.com/cypht-org/cypht/commit/6811cfc93651c09e670dc98de268b46d93b7ed7e) Added are you sure on forms if user setting is activated - [`f5c2b6b`](https://github.com/cypht-org/cypht/commit/f5c2b6b87c4631337d928aa1eeb13299c65a5df0) Fixed tests ### 📊 Changes **24 files changed** (+283 additions, -22 deletions) <details> <summary>View changed files</summary> 📝 `language/az.php` (+2 -2) 📝 `language/de.php` (+2 -1) 📝 `language/en.php` (+2 -1) 📝 `language/es.php` (+2 -1) 📝 `language/et.php` (+2 -1) 📝 `language/fa.php` (+2 -1) 📝 `language/fr.php` (+2 -1) 📝 `language/hu.php` (+2 -1) 📝 `language/id.php` (+2 -1) 📝 `language/it.php` (+2 -1) 📝 `language/ja.php` (+2 -1) 📝 `language/nl.php` (+2 -1) 📝 `language/pt-BR.php` (+2 -1) 📝 `language/ro.php` (+2 -1) 📝 `language/ru.php` (+2 -1) 📝 `language/zh-Hans.php` (+1 -0) 📝 `modules/core/handler_modules.php` (+17 -0) 📝 `modules/core/output_modules.php` (+23 -0) 📝 `modules/core/setup.php` (+4 -1) 📝 `modules/core/site.js` (+5 -1) _...and 4 more files_ </details> ### 📄 Description If someone closes browser, or tab and has unsaved changes to settings this attempts to warn him. Fixes: - https://github.com/jasonmunro/cypht/issues/523 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-25 21:36:49 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/cypht#971
No description provided.