-
Notifications
You must be signed in to change notification settings - Fork 4
DRAFT: Leaflet map using IFrame #9
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
base: main
Are you sure you want to change the base?
Conversation
In the last few commit I did:
|
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.
Thx for the PR
Did you test it? Does it work?
dist/client/index.html
Outdated
@@ -7,5 +7,5 @@ | |||
transition: opacity 0.25s 0.25s ease-in-out; | |||
} | |||
</style></head><body class="silex-loading"><div class="silex-loader silex-dialog">Loading | |||
</div><link rel="stylesheet" type="text/css" href="css/admin.css?938790"><main class="silex-workspace"><div class="silex-main silex-dialog-hide silex-ui-full" id="gjs"></div></main><script src="js/main.js?938790"></script><script>silex.start() | |||
</div><link rel="stylesheet" type="text/css" href="css/admin.css?216955"><main class="silex-workspace"><div class="silex-main silex-dialog-hide silex-ui-full" id="gjs"></div></main><script src="js/main.js?216955"></script><script>silex.start() |
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.
Did you change the favicon?
Why commit this?
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.
I didn't saw that I published this file. It modifies on his own every time I run my code so I didn't saw that the favicon had changed.
src/ts/client/grapesjs/index.ts
Outdated
@@ -364,6 +364,144 @@ export async function initEditor(config: EditorConfig) { | |||
try { | |||
/* @ts-ignore */ | |||
editor = grapesjs.init(config) | |||
|
|||
// Remove default map block and component |
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.
Everything bellow should be in a grapesjs plugin in a separate file
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.
It will be added in my next commit
src/ts/client/grapesjs/index.ts
Outdated
console.warn('Localisation not found') | ||
} | ||
} catch (error) { | ||
console.error('Erreur de géocodage:', error) |
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.
On parle français alors finalement ?
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.
Fixed in the next commit, sorry didn't saw.
src/ts/client/grapesjs/index.ts
Outdated
}, | ||
view: { | ||
onRender({ el }) { | ||
el.style.border = 'none' |
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.
Why here and not with the rest of the styles (width, height)
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.
I chose to keep border: none in view.onRender because I wanted to ensure it remains fixed and is not overridden, as it prevents the default browser border on iframes. Width and height are placed in defaults.style to allow users to modify them with the StyleManager. But I can still move it if you want
Hello I update the feature again.
I did add and change a lot of things:
Requesting feedback on the handleMessage implementation, guidance on fixing the persistence issue with map coordinates.