Skip to content

Commit f582c49

Browse files
committed
cleaned up styling on navbar again; there were mult overlapping style settings. Is now nested inside of state-container as it should be. Also added a console.log to background script import action to try to see what tabsObj looks like.
1 parent 5dd9e20 commit f582c49

File tree

2 files changed

+8
-21
lines changed

2 files changed

+8
-21
lines changed

src/app/styles/layout/_stateContainer.scss

Lines changed: 7 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -19,16 +19,6 @@
1919
box-shadow: none;
2020
}
2121

22-
.state-container .navbar {
23-
background-color: $background-color;
24-
display: flex;
25-
flex-direction: row;
26-
justify-content: flex-start;
27-
align-items: center;
28-
height: 30px;
29-
position: static;
30-
}
31-
3222
.state-container .main-navbar {
3323
background-color: $background-color;
3424
display: flex;
@@ -58,15 +48,6 @@
5848
height: 35px;
5949
}
6050

61-
.navbar {
62-
// prevent navbar from scrolling with state/tree display
63-
position: sticky;
64-
top: 40px; //JR 12.16.23 @12:15pm: this hardcoding is not best practice, should revisit to figure out a more dynamic way of sticking the navbar to the bottom of the main-navbar
65-
left: 0px;
66-
z-index: 1;
67-
@extend %disable-highlight;
68-
}
69-
7051
.no-data-message {
7152
color: $text-color;
7253
font: normal 13px $text-font-stack;
@@ -125,12 +106,17 @@
125106
}
126107

127108
.navbar {
128-
background-color: $navbar-color;
109+
background-color: $background-color;
129110
display: flex;
130111
flex-direction: row;
131-
justify-content: start;
112+
justify-content: flex-start;
132113
align-items: center;
133114
height: 30px;
115+
position: sticky;
116+
top: 40px; //JR 12.16.23 @12:15pm: this hardcoding is not best practice, should revisit to figure out a more dynamic way of sticking the navbar to the bottom of the main-navbar
117+
left: 0px;
118+
z-index: 1;
119+
@extend %disable-highlight;
134120
}
135121

136122
.main-navbar {

src/extension/background.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -207,6 +207,7 @@ chrome.runtime.onConnect.addListener((port) => {
207207
switch (action) {
208208
case 'import': // create a snapshot property on tabId and set equal to tabs object
209209
// may need do something like filter payload from stateless
210+
console.log('background import action tabsObj: ', tabsObj);
210211
tabsObj[tabId].snapshots = payload.snapshots; // reset snapshots to page last state recorded
211212
// tabsObj[tabId].hierarchy = savedSnapshot.hierarchy; // why don't we just use hierarchy? Because it breaks everything...
212213
tabsObj[tabId].hierarchy.children = payload.hierarchy.children; // resets hierarchy to last state recorded

0 commit comments

Comments
 (0)