-
Notifications
You must be signed in to change notification settings - Fork 2
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
chore: refactor eox-map
and convert code from ts
to js
#1206
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for eoxelements ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
…, layer, generate as js code
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.
This was a huge undertaking, thank you so much for this! On the first glance and in first integration tests it seems to work properly, but even if there should be any issues we can fix them at a later stage...
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.
Also from my side i have no real complaints, great work!
This is a ginormous PR (as expected), I only had a quick glance but it looks good to me.
if (newControls) { | ||
const keys = /** @type {ControlType[]} **/ Object.keys(controls); | ||
for (let i = 0, ii = keys.length; i < ii; i++) { | ||
const key = /** @type {ControlType} **/ keys[i]; | ||
|
||
// Add or update the control using the helper function | ||
// @ts-expect-error - Error throwing even type is declared. | ||
addOrUpdateControl(EOxMap, oldControls, key, controls[key]); | ||
} | ||
} |
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.
Unless I'm missing something, the only thing missing here are the round parenthesis ( )
around the variable to perform a proper type cast.
if (newControls) { | |
const keys = /** @type {ControlType[]} **/ Object.keys(controls); | |
for (let i = 0, ii = keys.length; i < ii; i++) { | |
const key = /** @type {ControlType} **/ keys[i]; | |
// Add or update the control using the helper function | |
// @ts-expect-error - Error throwing even type is declared. | |
addOrUpdateControl(EOxMap, oldControls, key, controls[key]); | |
} | |
} | |
if (newControls) { | |
const keys = /** @type {ControlType[]} **/ (Object.keys(controls)); | |
for (let i = 0, ii = keys.length; i < ii; i++) { | |
const key = keys[i]; | |
// Add or update the control using the helper function | |
addOrUpdateControl(EOxMap, oldControls, key, controls[key]); | |
} | |
} |
@@ -114,7 +141,7 @@ export class EOxMapCompare extends TemplateElement { | |||
() => html` | |||
<div class="eox-map-compare"> | |||
<div class="eox-map-compare__first"> | |||
<slot name="first"></slot> | |||
<slot name="first">hujhjjh</slot> |
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.
minor typo, it's spelled Juhuuu! (Hooray!) in German ;)
Implemented changes
eox-map
refactored version is released.Screenshots/Videos
Checklist before requesting a review