Skip to content
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

Restore live-reloading. Fix 404 bounce handler! #397

Merged
merged 1 commit into from Sep 20, 2022

Conversation

pablo-mayrgundter
Copy link
Member

This fixes the live-reload bug.

I bisected it down to cleanPlugin. No idea why that was causing trouble. Maybe related, there's no contents in the docs directory.. esbuild keeps the build in memory.

ALSO..... Fixed the 404 bounce to work locally. Not sure this ever worked before :(. I hadn't set the proxy up right.

We can finally reload any URL no problem!

@pablo-mayrgundter pablo-mayrgundter added the bug Something isn't working label Sep 17, 2022
@pablo-mayrgundter pablo-mayrgundter self-assigned this Sep 17, 2022
@netlify
Copy link

netlify bot commented Sep 17, 2022

Deploy Preview for bldrs-share ready!

Name Link
🔨 Latest commit a848e01
🔍 Latest deploy log https://app.netlify.com/sites/bldrs-share/deploys/63252d2ab485b40008af094b
😎 Deploy Preview https://deploy-preview-397--bldrs-share.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Copy link
Member

@OlegMoshkovich OlegMoshkovich left a comment

Choose a reason for hiding this comment

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

LGTM

@pablo-mayrgundter pablo-mayrgundter merged commit 656732b into bldrs-ai:main Sep 20, 2022
@pablo-mayrgundter pablo-mayrgundter deleted the live-reload branch September 20, 2022 13:16
pablo-mayrgundter added a commit to pablo-mayrgundter/Share that referenced this pull request Sep 22, 2022
OlegMoshkovich added a commit to OlegMoshkovich/Share that referenced this pull request Sep 23, 2022
Styling changes.  Fix local serving to match bldrs-ai#397.
OlegMoshkovich added a commit that referenced this pull request Sep 23, 2022
* squash

* update

* clean up

* properties container styles

* subtract tooltips for mobile

* re-order imports

* re-order the imports

* include scrollIcon in the prop row if the length of the property is too long

* add camera positions to the sample projects

* adjust logo camera position on mobile

* add bullet points to the about description

* test mobile alignment of the bullet in about control

* revert changes to about description

* change the font weightl of the key and value

* about and share dialogs

* re order imports

* Styling changes.  Fix local serving to match #397.

* Use CSS hover styling to reveal large text.

* space clean up

* space clean up

* theme adjustments

* align notes

* ItemProperties: restore table layout, with double-click for full-width wrapped text display of large text blobs.  Lighter separators between rows than accordion sections.  Lighter font weight overall.

* them line color and notes styles

* fix the about text sizes

* about variant

* small font tweak

* small font tweak

* about details

* style dividers

* adjust values

* rewire open models

* itemProperties: refactor to simplify row creation and react key allocation.

* add sample projects

* toggle notes, placed pointer curson on the table row

* add sample project

* add camera to sample

Co-authored-by: Pablo Mayrgundter <pablo.mayrgundter@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
No open projects
Status: Completed
Development

Successfully merging this pull request may close these issues.

None yet

3 participants