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

implemented remove individual tabs just for visible state

Browse source
This commit is contained in:
shivangikakkar 2018-10-21 23:24:18 +05:30
parent 1ea04587d9
commit c15eee22c6
2 changed files with 3 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/css/popup.css
View file

@ -682,12 +682,9 @@ span ~ .panel-header-text {
opacity: 1;
}
.container-info-tab-row:not(.clickable) .container-close-tab {
opacity: 0;
}
.container-info-tab-row:not(.clickable) .container-close-tab,
.container-info-tab-row:not(.clickable):hover .container-close-tab {
opacity: 0.5;
opacity: 0;
}
.container-info-has-tabs,

12
src/js/popup.js
View file

@ -769,7 +769,6 @@ Logic.registerPanel(P_CONTAINER_INFO, {
},
buildInfoTable(tabs) {
const identity = Logic.currentIdentity();
// For each one, let's create a new line.
const fragment = document.createDocumentFragment();
for (let tab of tabs) { // eslint-disable-line prefer-const
@ -779,7 +778,7 @@ Logic.registerPanel(P_CONTAINER_INFO, {
tr.innerHTML = escaped`
<td></td>
<td class="container-info-tab-title truncate-text" title="${tab.url}" >${tab.title}</td>
<td><img src="/img/container-close-tab.svg" style="width:50%;" id="close-tab" class="container-close-tab clickable" /></td>`;
<td><img src="/img/container-close-tab.svg" style="width:50%;" id="close-tab" class="container-close-tab" /></td>`;
tr.querySelector("td").appendChild(Utils.createFavIconElement(tab.favIconUrl));
document.getElementById("container-info-table").appendChild(fragment);
@ -787,15 +786,6 @@ Logic.registerPanel(P_CONTAINER_INFO, {
const closeTab = document.querySelector("#close-tab");
closeTab.setAttribute("id", tab.id);
if(identity.hasHiddenTabs) {
closeTab.addEventListener("mouseover",function() {
tr.classList.add("clickable");
});
closeTab.addEventListener("mouseout",function() {
tr.classList.remove("clickable");
});
}
Logic.addEnterHandler(closeTab, async function(e) {
await browser.tabs.remove(Number(e.target.id));
window.close();