github-mirror/discourse
2
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2024-11-23 15:05:24 +08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

FIX: Allow dismissing Discard Drafts modal via ESC (#12532)

Browse Source
This commit is contained in:
Penar Musaraj 2021-03-26 13:07:19 -04:00 committed by GitHub
parent edc0f61920
commit db7be947df
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
app/assets/javascripts/discourse/app/components/d-modal.js
View File

@ -51,7 +51,7 @@ export default Component.extend({
//only respond to events when the modal is visible
if (!this.element.classList.contains("hidden")) {
if (e.which === 27 && this.dismissable) {
next(() => $(".modal-header button.modal-close").click());
next(() => this.attrs.closeModal("initiatedByESC"));
}
if (e.which === 13 && this.triggerClickOnEnter(e)) {
@ -130,9 +130,7 @@ export default Component.extend({
this.set("dismissable", true);
}
if (data.headerClass) {
this.set("headerClass", data.headerClass);
}
this.set("headerClass", data.headerClass || null);
if (this.element) {
const autofocusInputs = this.element.querySelectorAll(

4
app/assets/javascripts/discourse/app/mixins/key-enter-escape.js
View File

@ -3,6 +3,10 @@ import { isiPad } from "discourse/lib/utilities";
// A mixin where hitting ESC calls `cancelled` and ctrl+enter calls `save.
export default {
keyDown(e) {
if (document.body.classList.contains("modal-open")) {
return;
}
if (e.which === 27) {
this.cancelled();
return false;

2
app/assets/javascripts/discourse/app/templates/modal/discard-draft.hbs
View File

@ -1,4 +1,4 @@
{{#d-modal-body dismissable=false headerClass="empty"}}
{{#d-modal-body headerClass="hidden"}}
<div class="instructions">
{{i18n "post.cancel_composer.confirm"}}
</div>

6
app/assets/stylesheets/common/base/modal.scss
View File

@ -54,10 +54,8 @@
.modal-header {
display: flex;
&:not(.empty) {
padding: 10px 15px;
border-bottom: 1px solid var(--primary-low);
}
padding: 10px 15px;
border-bottom: 1px solid var(--primary-low);
align-items: center;
.title {