Skip to content

Conversation

@rodmoreno
Copy link
Contributor

📑 Description

This PR introduces compatibility for the Conventional Comments extension with GitLab

✅ Checks

  • My pull request adheres to the code style of this project
  • I have updated the documentation if required
  • All the CI workflows have passed

ℹ️ Additional Information

Key changes:

  • Updated manifest-base.json to include *://gitlab.com/* in
    host_permissions and content_scripts.matches.
  • Modified content.js:
    • Added comments to distinguish between GitHub and GitLab selectors.
    • Updated GitLab CSS selectors to be more specific:
      • textarea[name="note[note]"] (for MR discussions, incl. line comments)
      • textarea[name="work-item-add-or-edit-comment"] (for Issue discussions, incl. new Issue descriptions & comments)

google-labs-jules bot and others added 2 commits May 21, 2025 14:37
This commit introduces compatibility for the Conventional Comments extension
with GitLab and refines the CSS selectors for improved accuracy and
maintainability.

Key changes:
- Updated `manifest-base.json` to include `*://gitlab.com/*` in
  `host_permissions` and `content_scripts.matches`.
- Modified `content.js`:
    - Added comments to distinguish between GitHub and GitLab selectors.
    - Updated GitLab CSS selectors to be more specific:
        - `textarea[name="note[note]"]` (for MR discussions, incl. line comments)
        - `textarea[name="work-item-add-or-edit-comment"]` (for Issue discussions, incl. new Issue descriptions & comments)
    - Removed previously used ID-based GitLab selectors.

The extension has been tested and confirmed to work on both GitLab and
GitHub after these changes.
align comments for selectors
@pullpo-for-slack
Copy link

pullpo-for-slack bot commented May 21, 2025

AI Analysis

Purpose of the changes:

The changes aim to extend the functionality of a browser extension to support GitLab in addition to GitHub, allowing users to interact with textareas for comments and discussions on both platforms.

Detailed file changes

(dropdown):
  • In file content.js:

    • Added GitLab textarea selectors to support comments in GitLab merge requests and issues.
    • Updated comments to clarify which selectors are for GitHub and which are for GitLab.
  • In file manifest-base.json:

    • Included GitLab in the host permissions to allow the extension to function on GitLab domains.
    • Updated content script matches to include GitLab URLs, enabling the extension's functionality on GitLab.

@mpl1018 mpl1018 self-requested a review May 23, 2025 08:08
@pullpo-for-slack
Copy link

🔗Pullpo.io Slack PR-Channel

@mpl1018
Copy link
Contributor

mpl1018 commented May 23, 2025

praise
This is good! Thanks @rodmoreno

@mpl1018 mpl1018 merged commit 1e23974 into pullpo-io:main May 23, 2025
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