Skip to content

Resized add to watch-list button. #460

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
Jun 15, 2022
Merged

Resized add to watch-list button. #460

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
ceb1547
Fixed linting errors
haydenmcp Jun 13, 2022
48555a0
Moved analysis and creation bar to base of summary page.
haydenmcp Jun 13, 2022
921b810
Added some summary page styling
haydenmcp Jun 13, 2022
83306fb
Removed center alignment of desktop cards
haydenmcp Jun 13, 2022
d42ab3d
Removed delayForPageRender import
haydenmcp Jun 13, 2022
c543f96
Merge branch 'master' of github.com:ThinkDeepTech/thinkdeep into hm/r…
haydenmcp Jun 13, 2022
9a1da19
Added hack to make button size correct on summary page.
haydenmcp Jun 15, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion packages/deep-card/deep-card.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ export class DeepCard extends LitElement {
}

:host(:hover) {
box-shadow: 0 0 8px var(--shadow-color, gray);
box-shadow: 0 0 var(--shadow-size, 12px) var(--shadow-color, gray);
}

.header ::slotted(*) {
Expand Down
19 changes: 14 additions & 5 deletions packages/deep-economic-analyzer/deep-analyzer-page-summary.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -115,9 +115,16 @@ export default class DeepAnalyzerPageSummary extends LitElement {
/**
* Lit callback executed on first update of the component.
*/
firstUpdated() {
async firstUpdated() {
super.firstUpdated();
this._setChartOptions();

// NOTE: TODO: While a fix goes into place allowing mwc-button height/width to be set this
// hack will be used to make the button size equal to what's desired for the app.
await this.updateComplete;
const materialButton = this.shadowRoot.querySelector('mwc-button');
const button = materialButton.shadowRoot.querySelector('#button');
button.setAttribute('style', 'height: 100%; width: 100%;');
}

/**
Expand Down Expand Up @@ -196,8 +203,8 @@ export default class DeepAnalyzerPageSummary extends LitElement {

.watch {
display: grid;
grid-template-columns: 80% 19%;
grid-gap: 1%;
grid-template-columns: 80% 19.65%;
grid-gap: 0.35%;
justify-content: center;
align-items: center;
}
Expand All @@ -214,6 +221,8 @@ export default class DeepAnalyzerPageSummary extends LitElement {
}

mwc-button {
height: 100%;
width: 100%;
--mdc-theme-primary: var(--primary-color-light);
--mdc-theme-on-primary: var(--secondary-color);
}
Expand Down Expand Up @@ -297,7 +306,7 @@ export default class DeepAnalyzerPageSummary extends LitElement {

<div class="input watch">
<mwc-textfield
label="i.e, Google"
label="Watch (i.e, Google)"
@input="${this._onInput.bind(this)}"
></mwc-textfield>
<mwc-button
Expand All @@ -309,7 +318,7 @@ export default class DeepAnalyzerPageSummary extends LitElement {

<mwc-select
class="input"
label="Analyze a business"
label="Analyze (i.e, Google)"
@selected="${this._onSelect}"
>
${this.configuration.observedEconomicEntities.map(
Expand Down