Skip to content

Commit 0b713c7

Browse files
authored
Resized add to watch-list button. (#460)
* Fixed linting errors * Moved analysis and creation bar to base of summary page. * Added some summary page styling * Removed center alignment of desktop cards * Removed delayForPageRender import * Added hack to make button size correct on summary page.
1 parent 07f9775 commit 0b713c7

File tree

2 files changed

+15
-6
lines changed

2 files changed

+15
-6
lines changed

packages/deep-card/deep-card.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ export class DeepCard extends LitElement {
2222
}
2323
2424
:host(:hover) {
25-
box-shadow: 0 0 8px var(--shadow-color, gray);
25+
box-shadow: 0 0 var(--shadow-size, 12px) var(--shadow-color, gray);
2626
}
2727
2828
.header ::slotted(*) {

packages/deep-economic-analyzer/deep-analyzer-page-summary.js

Lines changed: 14 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -115,9 +115,16 @@ export default class DeepAnalyzerPageSummary extends LitElement {
115115
/**
116116
* Lit callback executed on first update of the component.
117117
*/
118-
firstUpdated() {
118+
async firstUpdated() {
119119
super.firstUpdated();
120120
this._setChartOptions();
121+
122+
// NOTE: TODO: While a fix goes into place allowing mwc-button height/width to be set this
123+
// hack will be used to make the button size equal to what's desired for the app.
124+
await this.updateComplete;
125+
const materialButton = this.shadowRoot.querySelector('mwc-button');
126+
const button = materialButton.shadowRoot.querySelector('#button');
127+
button.setAttribute('style', 'height: 100%; width: 100%;');
121128
}
122129

123130
/**
@@ -196,8 +203,8 @@ export default class DeepAnalyzerPageSummary extends LitElement {
196203
197204
.watch {
198205
display: grid;
199-
grid-template-columns: 80% 19%;
200-
grid-gap: 1%;
206+
grid-template-columns: 80% 19.65%;
207+
grid-gap: 0.35%;
201208
justify-content: center;
202209
align-items: center;
203210
}
@@ -214,6 +221,8 @@ export default class DeepAnalyzerPageSummary extends LitElement {
214221
}
215222
216223
mwc-button {
224+
height: 100%;
225+
width: 100%;
217226
--mdc-theme-primary: var(--primary-color-light);
218227
--mdc-theme-on-primary: var(--secondary-color);
219228
}
@@ -297,7 +306,7 @@ export default class DeepAnalyzerPageSummary extends LitElement {
297306
298307
<div class="input watch">
299308
<mwc-textfield
300-
label="i.e, Google"
309+
label="Watch (i.e, Google)"
301310
@input="${this._onInput.bind(this)}"
302311
></mwc-textfield>
303312
<mwc-button
@@ -309,7 +318,7 @@ export default class DeepAnalyzerPageSummary extends LitElement {
309318
310319
<mwc-select
311320
class="input"
312-
label="Analyze a business"
321+
label="Analyze (i.e, Google)"
313322
@selected="${this._onSelect}"
314323
>
315324
${this.configuration.observedEconomicEntities.map(

0 commit comments

Comments
 (0)