- Issue created by @gallegosj
- π³π±Netherlands johnv
I think tisis a 'Works as designed' situation:
Q. Adding a comment to a day that has no hours removes the Empty day notation (i.e. "Closed") message from the View.
A. This is deliberate. The comment typically includes the closing-reason. I believe the comment is even translatable, d=so if you add 'Closed' to the comment, it will have the same translation as the preset one.Q. Additionally, for days that have hours, there is no space between the hours and the comment.
A. Please check the 'separators' section in the field settings. You can add a space there.Please reopen this ticket if yo think otherwise.
- π³π±Netherlands johnv
This is not only in Views, but also in the 'normal' node view.
Alternatively, what is your opinion on the following ready-to-run proposal: #3152911: Move 'Closed text' to Comment column if enabled β ?
- πΊπΈUnited States gallegosj
Q. Adding a comment to a day that has no hours removes the Empty day notation (i.e. "Closed") message from the View.
A. This is deliberate. The comment typically includes the closing-reason. I believe the comment is even translatable, d=so if you add 'Closed' to the comment, it will have the same translation as the preset one.I suppose this makes sense, but my site's stakeholders do not like it. Thanks for the explanation.
Q. Additionally, for days that have hours, there is no space between the hours and the comment.
A. Please check the 'separators' section in the field settings. You can add a space there.All of the separators are pre-populated in Views but appear to be ineffective. I too expected one of these to work, but none do.
- π³π±Netherlands johnv
I believe you use the 'Plain text' formatter, not the 'Table' formatter? Using the latter would solve the problem.
- π³π±Netherlands johnv
Te above commit solves your theming problem:
.office-hours__item-comments { /* Do not wrap label, slots, comments in cells. */ vertical-align: text-top; white-space: nowrap; + margin-left: 3px; /* Adds space before the element */ }
Automatically closed - issue fixed for 2 weeks with no activity.