mirror of https://github.com/zulip/zulip.git
message_edit: Eliminat whitespace in recipient row controls.
This is hacky, but I can't figure out another way to do it that doesn't cause other problems. Ideally, we'd add some sort of exclude rule to our HTML template linter so we can check the rest of the file.
This commit is contained in:
parent
3c8eb3c743
commit
b49448de7d
|
@ -29,9 +29,8 @@
|
|||
{{subject}}
|
||||
{{/if}}
|
||||
</a>
|
||||
</span>
|
||||
|
||||
<span class="recipient_bar_controls">
|
||||
<!-- The missing whitespace on the next line is a hack; ideally, would be user-select: none. -->
|
||||
</span><span class="recipient_bar_controls no-select">
|
||||
{{! edit subject pencil icon }}
|
||||
{{#if always_visible_topic_edit}}
|
||||
<i class="icon-vector-pencil always_visible_topic_edit"></i>
|
||||
|
|
|
@ -147,10 +147,20 @@ def check_handlebar_templates(templates):
|
|||
# type: (Iterable[str]) -> None
|
||||
# Check all our handlebars templates.
|
||||
templates = [fn for fn in templates if fn.endswith('.handlebars')]
|
||||
|
||||
IGNORE_FILES = [
|
||||
# TODO: Add some exclude mechanism for the line-wrapping issue here.
|
||||
'static/templates/recipient_row.handlebars',
|
||||
]
|
||||
|
||||
for fn in templates:
|
||||
if fn in IGNORE_FILES:
|
||||
continue
|
||||
validate(fn=fn, check_indent=True)
|
||||
|
||||
for fn in templates:
|
||||
if fn in IGNORE_FILES:
|
||||
continue
|
||||
if not validate_indent_html(fn):
|
||||
sys.exit(1)
|
||||
|
||||
|
|
Loading…
Reference in New Issue