Skip to content

Commit 2cdd123

Browse files
author
Marc Schlaeppi
committed
Fix Milkdown description modal layout
1 parent a884e77 commit 2cdd123

9 files changed

Lines changed: 11 additions & 34 deletions

File tree

source/app/blueprints/pages/case/templates/modal_add_case_event.html

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -79,13 +79,7 @@ <h4 class="modal-title mr-4">{% if event.event_id %} Event ID #{{ event.event_i
7979
<div class="row">
8080
<div class="form-group mt-3 col-12">
8181
<label for="event_content" class="placeholder">Event description</label>
82-
<div class="md_description_field">
83-
<div class="row" style="margin-left:0px;">
84-
<div class="col-12">
85-
<div id="event_description" contenteditable="true" spellcheck="true" class="mr-2" data-theme="{% if current_user.in_dark_mode %}dark{% else %}light{% endif %}">{% if event.event_content %}{{ event.event_content }}{% endif %}</div>
86-
</div>
87-
</div>
88-
</div>
82+
<div id="event_description" contenteditable="true" spellcheck="true" data-theme="{% if current_user.in_dark_mode %}dark{% else %}light{% endif %}">{% if event.event_content %}{{ event.event_content }}{% endif %}</div>
8983
</div>
9084
</div>
9185
</div>

source/app/blueprints/pages/case/templates/modal_add_case_ioc.html

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -72,13 +72,7 @@ <h4 class="modal-title mr-4">{% if ioc.ioc_id %}Edit IOC #{{ ioc.ioc_id }}{% els
7272
{% endif %}
7373
<div class="form-group mt-3">
7474
<label for="ioc_description" class="placeholder">Description</label>
75-
<div class="md_description_field">
76-
<div class="row" style="margin-left:0px;">
77-
<div class="col-12">
78-
<div id="ioc_description" contenteditable="true" spellcheck="true" class="mr-2" data-theme="{% if current_user.in_dark_mode %}dark{% else %}light{% endif %}">{% if ioc and ioc.ioc_description %}{{ ioc.ioc_description }}{% endif %}</div>
79-
</div>
80-
</div>
81-
</div>
75+
<div id="ioc_description" contenteditable="true" spellcheck="true" data-theme="{% if current_user.in_dark_mode %}dark{% else %}light{% endif %}">{% if ioc and ioc.ioc_description %}{{ ioc.ioc_description }}{% endif %}</div>
8276
</div>
8377
</div>
8478
<div class="form-group">

source/app/blueprints/pages/case/templates/modal_add_case_rfile.html

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -100,13 +100,7 @@ <h4 class="modal-title mr-4">{% if rfile.id %}Edit evidence #{{rfile.id}}{% else
100100

101101
<div class="form-group">
102102
<label for="rfile_desc" class="placeholder">Description</label>
103-
<div class="md_description_field">
104-
<div class="row" style="margin-left:0px;">
105-
<div class="col-12">
106-
<div id="evidence_description" contenteditable="true" spellcheck="true" class="mr-2" data-theme="{% if current_user.in_dark_mode %}dark{% else %}light{% endif %}">{% if rfile %}{{ rfile.file_description }}{% endif %}</div>
107-
</div>
108-
</div>
109-
</div>
103+
<div id="evidence_description" contenteditable="true" spellcheck="true" data-theme="{% if current_user.in_dark_mode %}dark{% else %}light{% endif %}">{% if rfile %}{{ rfile.file_description }}{% endif %}</div>
110104
</div>
111105
</div>
112106
<div class="form-group">

source/app/blueprints/pages/case/templates/modal_add_case_task.html

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -59,13 +59,7 @@ <h4 class="modal-title mr-4">{% if task.id %} Task ID #{{ task.id }}{% else %}
5959
</div>
6060
<div class="form-group mt-3">
6161
<label for="task_description" class="placeholder">Description</label>
62-
<div class="md_description_field">
63-
<div class="row" style="margin-left:0px;">
64-
<div class="col-12">
65-
<div id="task_description" contenteditable="true" spellcheck="true" class="mr-2" data-theme="{% if current_user.in_dark_mode %}dark{% else %}light{% endif %}">{% if task.task_description %}{{ task.task_description }}{% endif %}</div>
66-
</div>
67-
</div>
68-
</div>
62+
<div id="task_description" contenteditable="true" spellcheck="true" data-theme="{% if current_user.in_dark_mode %}dark{% else %}light{% endif %}">{% if task.task_description %}{{ task.task_description }}{% endif %}</div>
6963
</div>
7064
</div>
7165
<div class="form-group mt-3">

ui/src/pages/case.ioc.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ function add_ioc() {
3333
$('#last_saved > i').attr('class', "fa-solid fa-file-circle-exclamation");
3434
}, null);
3535

36-
g_ioc_desc_editor.setOption("minLines", "10");
36+
g_ioc_desc_editor.setOption("minLines", "6");
3737
$('#submit_new_ioc').on("click", function () {
3838
if(!$('form#form_new_ioc').valid()) {
3939
return false;
@@ -171,7 +171,7 @@ function edit_ioc(ioc_id) {
171171
$('#last_saved > i').attr('class', "fa-solid fa-file-circle-exclamation");
172172
}, null);
173173

174-
g_ioc_desc_editor.setOption("minLines", "10");
174+
g_ioc_desc_editor.setOption("minLines", "6");
175175

176176
load_menu_mod_options_modal(ioc_id, 'ioc', $("#ioc_modal_quick_actions"));
177177
$('.dtr-modal').hide();

ui/src/pages/case.rfiles.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ function add_modal_rfile() {
4141
$('#last_saved').addClass('btn-danger').removeClass('btn-success');
4242
$('#last_saved > i').attr('class', "fa-solid fa-file-circle-exclamation");
4343
}, null);
44-
g_evidence_desc_editor.setOption("minLines", "10");
44+
g_evidence_desc_editor.setOption("minLines", "6");
4545
load_evidence_type();
4646

4747
$('#modal_add_rfiles').modal({ show: true });

ui/src/pages/case.tasks.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ function add_task() {
1818
$('#last_saved').addClass('btn-danger').removeClass('btn-success');
1919
$('#last_saved > i').attr('class', "fa-solid fa-file-circle-exclamation");
2020
}, null);
21-
g_task_desc_editor.setOption("minLines", "10");
21+
g_task_desc_editor.setOption("minLines", "6");
2222
$('#submit_new_task').on("click", function () {
2323

2424
clear_api_error();

ui/src/pages/case.timeline.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ function add_event(parent_event_id = null) {
2020
$('#last_saved > i').attr('class', "fa-solid fa-file-circle-exclamation");
2121
}, null);
2222

23-
g_event_desc_editor.setOption("minLines", "10");
23+
g_event_desc_editor.setOption("minLines", "6");
2424
let parent_selector = $('#parent_event_id');
2525

2626
// Add empty option

ui/test/legacy_description_controls.test.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,8 @@ test('case description forms use Milkdown without legacy preview controls', asyn
1818
);
1919
const script = await readFile(new URL(`ui/src/pages/${scriptName}`, root), 'utf8');
2020

21-
assert.doesNotMatch(template, new RegExp(`${prefix}_preview_button|icon-note`));
21+
assert.doesNotMatch(template, new RegExp(`${prefix}_preview_button|icon-note|md_description_field`));
2222
assert.doesNotMatch(script, new RegExp(`edit_in_${prefix}_desc|preview_${prefix}_description`));
23+
assert.doesNotMatch(script, new RegExp(`g_${prefix}_desc_editor\\.setOption\\("minLines", "10"\\)`));
2324
}
2425
});

0 commit comments

Comments
 (0)