Skip to content

Conversation

@dalekunce
Copy link
Member

@dalekunce dalekunce commented Oct 3, 2025

Working on some clean up from the last push.

Dale Kunce added 11 commits October 3, 2025 07:12
- Add mobile-menu.js with touch/click event handling for mobile devices
- Replace CSS-only hover behavior with JavaScript-based menu toggle
- Add touch-friendly styling with proper cursor and tap highlight removal
- Implement menu auto-close on outside click, escape key, and window resize
- Ensure mobile menu works properly on devices where hover doesn't function
- Mobile menu activates at screen width < 650px as per existing breakpoints
- Remove i18n_filter.rb plugin and get_locale.html include (no longer needed)
- Replace all localize filter usage with Jekyll Polyglot approach
- Fix blog post date localization using site.data[site.active_lang].months array
- Update month data structure with null placeholder for 1-based indexing
- Replace get_locale.html includes with direct locale = site.active_lang assignment
- All date localization now works correctly across EN/FR/ES/CS languages
- Blog posts and listings display proper localized month names
- Mobile menu JavaScript functionality preserved and working
- Add post grouping logic to avoid duplicate posts on same day
- Include language links for posts available in multiple languages
- Add translation strings for 'also available in' text across all languages
- Style language links with professional appearance
- Maintain backward compatibility with single-language posts
- Add events.json to .gitignore since it's generated during build
- Add .jekyll-metadata to .gitignore since it changes on every build
- Remove these files from git tracking while keeping them in working directory
- This prevents merge conflicts on files that are automatically regenerated
…ated

- Keep events.json deleted since it's now in .gitignore
- File will be generated fresh during build process
- Prevents future merge conflicts on auto-generated content
- Auto-fix indentation, spacing, and code style issues

Resolves CI build failure due to linting errors
- Update ESLint from 8.x to 9.x to resolve deprecation warnings
- Update glob from 7.x to 10.x for better performance and security
- Update js-yaml from 3.x to 4.x for latest security patches
- Update node-fetch from 2.x to 3.x with dynamic import support
- Replace gulp-clean with modern 'del' package to avoid rimraf issues
- Remove unused glob-all dependency
- Fix unused error variables in events.js for ESLint compliance
- Update fetch-events.cjs to work with node-fetch v3 ESM format

All builds and linting now pass without warnings
- del package is now ESM-only and requires dynamic imports in CommonJS files
- Update actions/checkout from v4 to v5 across all workflows
- Keep upload/download-artifact at v4 (v5 not yet available)
- Configure pagination to display 15 posts per page with proper URL structure
- Add pagination navigation with Previous/Next buttons and page info

Closes #411
- Improved mobile blog layout with responsive cards and pagination
- Added mobile-first breakpoints (768px, 480px) throughout
- Increased button touch targets
- Enhanced mobile footer and landing page responsiveness
- Improved mobile typography scaling and spacing
- Added better mobile menu interactions and accessibility features
- All components now fully responsive with touch-friendly interfaces
Dale Kunce added 2 commits October 3, 2025 17:51
- Replace special characters in French URLs to prevent encoding issues
- Fix ESLint errors in mobile-menu.js: spacing and formatting
@dalekunce dalekunce mentioned this pull request Oct 3, 2025
…racking

- events.json should be auto-generated, not tracked in git
- keeps .gitignore entry to prevent future tracking
- resolves divergent branch issue between modernize-2025 and publish
@dalekunce dalekunce marked this pull request as ready for review October 5, 2025 23:36
@dalekunce dalekunce merged commit 5601622 into publish Oct 5, 2025
4 checks passed
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.

2 participants