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.
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
Devtools fixes #54912
Devtools fixes #54912
Conversation
75c4bf0
to
12a8922
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for putting this together and apologies for the review delay. No major concerns, I think you just dropped a file in your stopPropagation
commit.
@AleksanderBodurri, can you take a quick look as well just to make sure I'm not missing anything?
@@ -344,6 +344,10 @@ export class DirectiveForestComponent { | |||
} | |||
} | |||
|
|||
stopPropagation(event: Event): void { | |||
event.stopPropagation(); | |||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Question: Is this used anywhere? Was this referenced in the template somewhere and dropped from this commit?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah I'm not sure why the change was missing, It's back now.
<mat-icon matTooltip="Skipped" class="hydration skipped">invert_colors_off</mat-icon> | ||
} | ||
@case ('mismatched') { | ||
<mat-icon matTooltip="Mismatch" class="hydration mismatched">error_outline</mat-icon> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggestion: One of my points of feedback was that it's not super clear what was "Mismatched", especially for users not that familiar with hydration. Should we update these tool tips to actually use the word "hydration" such as "Hydration skipped" and "Hydration mismatch"? That way a developer would have something meaningful to search for if they want to understand what exactly this means.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
My one other minor suggestion is a brief comment either on (dblclick)
or stopPropogation
about the action it's trying to prevent, since it's purpose doesn't seem super obvious to me.
@JeanMeche the merge tooling failed to process this PR. Could you please rebase on top of the most recent |
…ective forest In some cases the height of the viewport wasn't calculated correctly because of extension tabs quirks. This commit fixes this issue. Fixes angular#53704
In some cases signals weren't unwraped thus not reading the value correctly. This commit fixes this issue.
b67d98e
to
796c2f2
Compare
…element panel This will improve the UX
…ollapsed This commit improves the devtools UX.
796c2f2
to
0b1c58e
Compare
We should be ✅ now |
This PR was merged into the repository by commit 96972b4. |
In some cases signals weren't unwraped thus not reading the value correctly. This commit fixes this issue. PR Close #54912
In some cases signals weren't unwraped thus not reading the value correctly. This commit fixes this issue. PR Close #54912
In some cases signals weren't unwraped thus not reading the value correctly. This commit fixes this issue. PR Close #54912
See individual commits.
fixes #53704