Skip to content

feat: rewrite the Editor the Rails way #186

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 128 commits into
base: master
Choose a base branch
from
Open

feat: rewrite the Editor the Rails way #186

wants to merge 128 commits into from

Conversation

did
Copy link
Contributor

@did did commented Aug 21, 2025 •
edited
Loading

  • setup Tailwindcss 4 (isolation mode) + Hotwire (Turbo + Stimulus) + ViewComponents + Lookbook
  • remove complete dependency to NodeJS
  • rewrite the app / drawer layouts (while keeping the animations)
  • manage pages (CRUD)
  • image gallery (CRUD)
  • manage sections (CRUD) with dynamic section/block forms
  • fix live editing feature
  • handle StaleRecord errors
  • implement the publish feature
  • AdminUI (for developers)

did added 30 commits July 26, 2025 13:34
@did
chore: setup Lookbook + ViewComponent + Tailwindcss 4 (without Node)
4edd06c
@did
chore: /editor -> /legacy-editor
a7ecf84
@did
chore: setup importmap (scoped for Maglev)
34119cd
@did
feat: mimic the UX of the legacy drawer
de2c552
@did
chore: sidebar OK + native drawer transition
d30e88b
@did
chore: pimp the IconComponent (+ Lookbook) + PageInfo component (WIP)…
fc6cbea 
… + TopBar component (WIP)
@did
chore: clean importmap config + Dropdown component
aa3bddd
@did
feat: new IconButton component
46f61aa
@did
feat: implement the MenuDropdown
1e8d431
@did
fix: the dropdown works better in fixed position
d7437d4
@did
feat: new AppLayout component
80a677f
@did
feat: Edit page screen (WIP)
7964740
@did
feat: new Tab component
37a817b
@did
feat: Edit page form works
1e5c8d8
@did
feat: smooth the XP when editing a page + new Badge component + impro…
53894c0 
…ve a lot of components
@did
chore: don't cache the page form + translate the flash messages of th…
691b486 
…e pages
@did
feat: new Checkbox component
5e37047
@did
feat: implement the Textarea component
3306414
@did
chore: refactor the IconComponent (was overengineered)
c40585d
@did
feat: button to clone the current page
bfeee27
@did
feat: copy live url to clipboard + use partials to render both the to…
93dcf4b 
…pbar and sidebar (easier customization)
@did
feat: new DeviceToggler component
0ffca8f
@did
feat: new LocaleSwitcher component
14a2fd5
@did
feat: new Modal component
b865eac
@did
feat: new ImageLibrary::Card component
8db59b0
@did
feat: new Pagination component
980ac67
@did
feat: new SearchForm component
8d0f7ae
@did
feat: ImageLibrary view (WIP)
8e43919
@did
feat: the ImageLibrary view is now complete
0d937a5
@did
feat: implement the ImageField component
853515b
did added 30 commits September 16, 2025 12:13
@did
feat: new CollectionItem component for the DynamicForm
901aee1 
chore: extra_fields -> spread_fields (option name)
@did
feat: move the caret to the end when focusing text inputs
497e8b4 
chore: Rubocop ✅
@did
feat: new Divider/Hint components for the Dynamic form
51d71bf
@did
feat: new IconField components
091c695
@did
feat: add an Advanced tab for advanced settings
07c88c0 
chore: clean I18n keys (WIP)
chore: clean the architecture of the Setting components
@did
chore: Rubocop ✅
edaa62c 
fix: fix the Link component (bad refactoring)
fix: fix wrong i18n keys (WIP)
@did
feat: implement the editing of a tree structure of blocks (requires s…
2d67d4e 
…ome cleaning)
@did
feat: display the block position within its parent
44f2e94 
chore: Rubocop ✅
chore: clean the templates of the list of section blocks
@did
fix: fix the CI for Rails 7.2
e11037e
@did
feat: skeleton for the Publish action
bda1006 
fix: fix a couple of minor issues
feat: display an empty message if no sections (WIP)
@did
feat: translate the Editor UI in the other locales
e41b03f
@did
fix: attempt to fix an installation with Propshaft
b99a497
@did
fix: still trying to fix a propshaft installation
1b4db0f
@did
fix: still trying to fix a propshaft installation
6b2536f
@did
fix: still trying to fix a propshaft installation
5123411
@did
fix: make it compliant with Ruby 3.2
c476fe4 
fix: the empty message is now correctly displayed
fix: don't display a blank page title if the page hasn't been translated yet
@did
fix: wrong tab label for the section form
8274713 
fix: unable to set the very first icon
@did
fix: fix some Hotwire glitches when coming back to old pages (modal +…
de19999 
… richtext)
@did
fix: moving a section from bottom of the list to the up wasn't working
9a2c7b8 
fix: fix minor UI issues
@did
feat: deal with the insert_at property when inserting a section
826ba01 
feat: optimistic locking enabled for sections, blocks and style
@did
feat: implement the singleton property of a section
b263887 
chore: don't update the page if the section is site scoped
@did
chore: add a modal for NotAuthorized error
d7e95a9 
chore: translate the error modals
@did
fix: couldn't update site-scoped sections or section blocks
8ea6379 
feat: display an error modal for ApplicationError (unless it's local)
@did
fix: still fixing errors by wrong lock versions (all related to the s…
9a229bd 
…ite scoped sections)
@did
chore: handle errors in a smooth way when the section or the block do…
fb2ed46 
…esn't exist anymore
@did
chore: raise a ActiveRecord::StaleObjectError exception if the sectio…
0a9176c 
…n or the block doesn't exist
@did
feat: refresh the section preview if it's changed
6d452dd
@did
fix: when browsing pages, update the paths used when clicking on a se…
8f7b3e6 
…ction setting (live preview)
@did
chore: make sure the Maglev client JS code is always executed in the …
5f670ba 
…iframe
@did
chore: set the lock version of the section on the client side
ac87748
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@did