Commit 565f39bb authored by Daniel Classon's avatar Daniel Classon Committed by Commit Bot

[OsSettingsPeoplePage] Change user list button to use dom-if

Change the remove restricted user button to use dom-if instead of
hidden. Relevant to deep linking as its easier to deep link to visible
buttons if hidden buttons aren't rendered.

Bug: 1084154
Change-Id: I831dc8f69ab00ffc51771a327b2fdf8ea47ab559
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2363551Reviewed-by: default avatarKyle Horimoto <khorimoto@chromium.org>
Commit-Queue: Daniel Classon <dclasson@google.com>
Cr-Commit-Position: refs/heads/master@{#799311}
parent 9e9707fd
......@@ -60,10 +60,12 @@
[[item.displayEmail]]
</div>
</div>
<cr-icon-button class="icon-clear"
hidden="[[shouldHideCloseButton_(disabled, item.isOwner)]]"
title="[[getRemoveUserTooltip_(item)]]" on-click="removeUser_">
</cr-icon-button>
<template is="dom-if"
if="[[!shouldHideCloseButton_(disabled, item.isOwner)]]">
<cr-icon-button class="icon-clear"
title="[[getRemoveUserTooltip_(item)]]" on-click="removeUser_">
</cr-icon-button>
</template>
</div>
</template>
</div>
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment