Skip to content

Avoid the use of navigator on server side#58

Open
DovahBrownies wants to merge 1 commit into
aarondail:masterfrom
DovahBrownies:master
Open

Avoid the use of navigator on server side#58
DovahBrownies wants to merge 1 commit into
aarondail:masterfrom
DovahBrownies:master

Conversation

@DovahBrownies

Copy link
Copy Markdown

Pretty straightforward;

navigator will obviously throw an error on the server side due to not existing outside the browser. This is a simple work around.

@kevinreber kevinreber left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure how active the owner is, but LGTM!

@stormynight9

Copy link
Copy Markdown

@DovahBrownies Have you published a new package with this fix included? If yes I really appreciated it you would share it with us!

@DovahBrownies

Copy link
Copy Markdown
Author

@DovahBrownies Have you published a new package with this fix included? If yes I really appreciated it you would share it with us!

This package didn't perform as smoothly as I had initially wanted, for my specific needs. So I didn't end up using it at all.

@hfossli-agens

Copy link
Copy Markdown

What did you use instead of this package @DovahBrownies?

@DovahBrownies

Copy link
Copy Markdown
Author

What did you use instead of this package @DovahBrownies?

I ended up creating my own component that behaves exactly the way I want it to.

@hfossli-agens

hfossli-agens commented Jun 5, 2025 via email

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants