Skip to content

refactor: remove jQuery dependencies and convert to vanilla JavaScript#2563

Merged
jacopen merged 1 commit intomainfrom
feature/remove-jquery-dependencies
Jun 22, 2025
Merged

refactor: remove jQuery dependencies and convert to vanilla JavaScript#2563
jacopen merged 1 commit intomainfrom
feature/remove-jquery-dependencies

Conversation

@jacopen
Copy link
Collaborator

@jacopen jacopen commented Jun 22, 2025

Summary

  • Convert jQuery DOM manipulation to native insertAdjacentHTML in chat functionality
  • Remove commented out jQuery code from talks.js
  • Eliminate all jQuery dependencies from the codebase

Changes

  • Chat functionality: $('#chat').append()document.getElementById('chat').insertAdjacentHTML('beforeend', ...)
  • Talks module: Removed commented out jQuery code for Bootstrap Table

Benefits

  • ✅ Reduced bundle size by eliminating jQuery dependency
  • ✅ Improved performance with native DOM APIs
  • ✅ Simplified dependency management
  • ✅ Better maintainability with modern JavaScript

Test Results

  • ✅ JavaScript build successful
  • ✅ All RSpec tests passing (837 examples, 0 failures)
  • ✅ RuboCop linting passed
  • ✅ Chat functionality verified working with vanilla JS

🤖 Generated with Claude Code

- Convert jQuery DOM manipulation to native insertAdjacentHTML in chat functionality
- Remove commented out jQuery code from talks.js
- Eliminate all jQuery dependencies from the codebase

The chat functionality now uses standard DOM APIs:
- `$('#chat').append()` → `document.getElementById('chat').insertAdjacentHTML('beforeend', ...)`

This change removes the last jQuery usage from the codebase, improving
bundle size and reducing external dependencies.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@gitops-for-cloudnativedays gitops-for-cloudnativedays bot added the reviewapps Build ReviewApp environment automatically if this label is granted label Jun 22, 2025
gitops-for-cloudnativedays bot added a commit to cloudnativedaysjp/dreamkast-infra that referenced this pull request Jun 22, 2025
commit: cloudnativedaysjp/dreamkast@4e2d0b7
action URL: https://github.com/cloudnativedaysjp/dreamkast/actions/runs/15808322696

Co-authored-by: gitops-for-cloudnativedays[bot] <113280573+gitops-for-cloudnativedays[bot]@users.noreply.github.com>
@github-actions
Copy link

@github-actions
Copy link

Simplecov Report

Covered Threshold
64.42% 60%

@jacopen jacopen merged commit 050e2b7 into main Jun 22, 2025
9 checks passed
@jacopen jacopen deleted the feature/remove-jquery-dependencies branch June 22, 2025 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

reviewapps Build ReviewApp environment automatically if this label is granted

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant