Simplify showing/hiding attached file name

Note we have to render the `<p>` tag in one line because at the time of
writing browsers don't consider elements with whitespace inside as empty
[1].

[1] https://developer.mozilla.org/en-US/docs/Web/CSS/:empty
This commit is contained in:
Javi Martín
2021-06-20 16:16:39 +02:00
parent afbd1fec37
commit 8116e75aee
5 changed files with 6 additions and 10 deletions

View File

@@ -92,7 +92,6 @@
}, },
clearFilename: function(data) { clearFilename: function(data) {
$(data.fileNameContainer).text(""); $(data.fileNameContainer).text("");
$(data.fileNameContainer).hide();
}, },
clearInputErrors: function(data) { clearInputErrors: function(data) {
$(data.errorContainer).find("small.error").remove(); $(data.errorContainer).find("small.error").remove();
@@ -102,7 +101,6 @@
}, },
setFilename: function(data, file_name) { setFilename: function(data, file_name) {
$(data.fileNameContainer).text(file_name); $(data.fileNameContainer).text(file_name);
$(data.fileNameContainer).show();
}, },
setProgressBar: function(data, klass) { setProgressBar: function(data, klass) {
$(data.progressBar).find(".loading-bar").addClass(klass); $(data.progressBar).find(".loading-bar").addClass(klass);

View File

@@ -92,7 +92,6 @@
}, },
clearFilename: function(data) { clearFilename: function(data) {
$(data.fileNameContainer).text(""); $(data.fileNameContainer).text("");
$(data.fileNameContainer).hide();
}, },
clearInputErrors: function(data) { clearInputErrors: function(data) {
$(data.errorContainer).find("small.error").remove(); $(data.errorContainer).find("small.error").remove();
@@ -105,7 +104,6 @@
}, },
setFilename: function(data, file_name) { setFilename: function(data, file_name) {
$(data.fileNameContainer).text(file_name); $(data.fileNameContainer).text(file_name);
$(data.fileNameContainer).show();
}, },
setProgressBar: function(data, klass) { setProgressBar: function(data, klass) {
$(data.progressBar).find(".loading-bar").addClass(klass); $(data.progressBar).find(".loading-bar").addClass(klass);

View File

@@ -42,6 +42,10 @@
.file-name { .file-name {
margin-top: 0; margin-top: 0;
&:empty {
display: none;
}
} }
.loading-bar { .loading-bar {

View File

@@ -17,9 +17,7 @@
</div> </div>
<div class="small-6 column"> <div class="small-6 column">
<p class="file-name"> <p class="file-name"><%= document_attachment_file_name %></p>
<%= document_attachment_file_name %>
</p>
</div> </div>
<div class="small-12 column"> <div class="small-12 column">

View File

@@ -19,9 +19,7 @@
</div> </div>
<div class="small-6 column"> <div class="small-6 column">
<p class="file-name"> <p class="file-name"><%= image_attachment_file_name %></p>
<%= image_attachment_file_name %>
</p>
</div> </div>
<div class="small-12 column"> <div class="small-12 column">