rowan/multi-account-containers
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix #142: disable arrow if container has no tabs (#153)

Browse source
This commit is contained in:
luke crouch 2017-02-07 11:15:54 -06:00 • committed by Jonathan Kingston
parent 4b3892f38e
commit 6115b1bcb9
2 changed files with 10 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
webextension/css/popup.css
View file

@ -274,6 +274,12 @@ table.unstriped tbody tr {
--identity-icon: url("/img/usercontext.svg#circle");
}
.no-tabs-img {
block-size: 12px;
filter: opacity(30%);
inline-size: 12px;
}
.show-tabs-img {
block-size: 12px;
inline-size: 12px;

6
webextension/js/popup.js
View file

@ -222,6 +222,8 @@ Logic.registerPanel(P_CONTAINERS_LIST, {
const fragment = document.createDocumentFragment();
Logic.identities().forEach(identity => {
const hasTabs = (identity.hasHiddenTabs || identity.hasOpenTabs);
const arrowClass = hasTabs ? "show-tabs-img" : "no-tabs-img";
const tr = document.createElement("tr");
fragment.appendChild(tr);
tr.classList.add("container-panel-row", "clickable");
@ -233,7 +235,7 @@ Logic.registerPanel(P_CONTAINERS_LIST, {
</div>
</td>
<td class="open-newtab">${identity.name}</td>
<td class="show-tabs"><img src="/img/container-arrow.svg" class="show-tabs show-tabs-img" /></td>`;
<td class="show-tabs"><img src="/img/container-arrow.svg" class="show-tabs ${arrowClass}" /></td>`;
tr.addEventListener("click", e => {
if (e.target.matches(".open-newtab")) {
@ -250,7 +252,7 @@ Logic.registerPanel(P_CONTAINERS_LIST, {
}).catch(() => {
window.close();
});
} else if (e.target.matches(".show-tabs")) {
} else if (hasTabs && e.target.matches(".show-tabs")) {
Logic.showPanel(P_CONTAINER_INFO, identity);
}
});