Merged
Conversation
Add activerecord model name translation for Tutorial (singular: "Video", plural: "Video Gallery") and replace all hardcoded references with model_name.human calls. Update nav icons from circle-info to circle-play. Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
The landing page was already labeled "Home" in navigation links. This aligns controllers, policy, views, routes, and specs with that naming: DashboardController → HomeController, DashboardPolicy → HomePolicy, namespace :dashboard → :home. Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
…ow (#1135) * Split event description: first part beside image, rest full width below Add split_rich_text helper that splits ActionText content at a block-element boundary based on character length. The event show page now displays the first ~800 characters of the description in the left column beside the image, with the remaining content rendered full width below the grid. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix split_rich_text for plain text content and normalize indentation When ActionText content has no block elements (e.g. plain text without <p> tags), element_children returns empty. Now falls back to returning the full HTML as desc_top. Also converts guard clause to one-liner for rubocop style compliance. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix rubocop SpaceInsideArrayLiteralBrackets offenses Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
* Add TomSelect person dropdowns with email display across all forms and search filters Replace all person/user select fields with searchable TomSelect dropdowns that display name in bold with email in gray parentheses. Convert free-text person search on workshop_ideas, bookmarks, and event_registrations indexes to dropdown filters while keeping legacy URL params (e.g. author_name) working. Sort all dropdowns case-insensitively by first name, last name, email. Fix nil registrant crash on event_registrations index. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Rename @people to @users where the variable holds User records The variable fed into a :user_id select field but was misleadingly named @people. Renamed in organizations_controller, workshop_logs_controller, and the workshop_logs search_boxes partial. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix registrant FK mismatch and expand dropdown sort order Change event_registrations @registrants from User to Person records since registrant_id is a FK to people.id, not users.id. Expand sort order on all person/user dropdowns to include all email fields: person.first_name, person.last_name, user.email, person.email_2, person.email. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Remove @registrants (using remote-select) and revert TomSelect CSS overrides Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Revert org form changes, keep @people→@users rename in show Reverts @people_array addition in set_form_variables and TomSelect wrapper in affiliation partial. Keeps @people→@users rename in show action for consistency. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Revert event_registrations form to main, restore @registrants in index The form uses remote-select from main. The search partial still needs @registrants so add it back as a Person query with standard sort order. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Rename @authors to @users in workshop_ideas controller and views Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix workshop_ideas view specs for @authors→@users rename and standardize search submit Assign @users in view specs. Switch workshop_logs search to use Stimulus submit action instead of inline onchange. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
* Redesign event show page layout and form with autoshow fields
Restructure event show page with centered header image, title (Lato font,
green), date/time (Lato, dark navy, uppercase), and registration section.
Add autoshow boolean fields for conditional display of event details.
Add timezone-aware datetime display with user timezone support (PST default).
Style register/deregister buttons with custom color (rgb(170,46,0)) and
Telefon Bold font. Extract videoconference link into turbo-replaceable partial.
Support up to 5 registration section instances for embedding in rich text.
Add calendar link location/videoconference URL fields.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* Add pre_title field and refine event form layout
Add pre_title string column to events with its own migration. Update form
with title card, pre-title field, and admin Display settings card. Show
pre_title on event show page in Telefon Bold. Comment out header_image
blob check in display_image pending rhino embed investigation.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* Fix Rubocop Omakase array bracket spacing
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* Update tests to match redesigned event show page
- Styled times format: full day/month names instead of abbreviated
- "Registration closes" instead of "Registration Close Date"
- "Join on {domain}" instead of "Join online"
- Remove "All times displayed in" assertions (text removed from show)
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---------
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
PDF previews were using object-cover which cropped the top and bottom of portrait-oriented pages. Switch to object-contain so the entire first page is visible in resource and home index cards. Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Preview renders the show page with unsaved form changes in a new tab, using assign_attributes (no DB writes). Includes preview banner, authorization via edit? policy alias, and request/policy specs. Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What is the goal of this PR and why is this important?
How did you approach the change?
Anything else to add?