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

refactor: Custom dialogs and file structure #9

Merged
merged 13 commits into from
Jan 23, 2021
Merged

Conversation

pklaschka
Copy link
Member

  1. Use custom dialog implementation to increase compatibility since, for now, the <dialog> element doesn't provide the compatibility we need
  2. Revamp file structure to make long files shorter and more readable

Closes fliegwerk/fliegdoc#26

1. Use custom dialog implementation to increase compatibility since, for now, the <dialog> element doesn't provide the compatibility we need
2. Revamp file structure to make long files shorter and more readable

Refs: fliegwerk/fliegdoc#26
@pklaschka pklaschka added 🐛 bug Something isn't working 🌷 enhancement New feature or request labels Jan 23, 2021
@pklaschka pklaschka requested a review from fussel178 January 23, 2021 01:32
@pklaschka pklaschka self-assigned this Jan 23, 2021
src/components/dialog-content.tsx Outdated Show resolved Hide resolved
src/hooks/use-on-submit-callback.tsx Outdated Show resolved Hide resolved
src/hooks/use-on-change-callback.tsx Outdated Show resolved Hide resolved
src/lib/open-page.tsx Outdated Show resolved Hide resolved
src/components/dialog-content.tsx Outdated Show resolved Hide resolved
src/components/form.tsx Show resolved Hide resolved
src/components/dialog-content.tsx Show resolved Hide resolved
@codeclimate
Copy link

codeclimate bot commented Jan 23, 2021

Code Climate has analyzed commit 201f249 and detected 0 issues on this pull request.

View more on Code Climate.

Copy link
Member

@fussel178 fussel178 left a comment

Choose a reason for hiding this comment

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

Have tested it with the current build of fliegdoc.
On desktop it works fine but in iOS it works on the main page of the fliegdoc documentation but not on the module pages (or "subpages"). (see attached video file)
It that a bug in fliegdoc or in the search-popup? 🤔

(tested with an iPhone 11)

Update

Nevermind, my fault. I forgot to replace the bundle.js in the subpage 😛.

IMG_0005.mp4

@pklaschka pklaschka merged commit be29f76 into main Jan 23, 2021
@pklaschka pklaschka deleted the refactor-and-compat branch January 23, 2021 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working 🌷 enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unable to close search on mobile
2 participants