Skip to content

ui fix for Selected repos #529

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

Open
JaYRaNa213 wants to merge 2 commits into
base: main
Choose a base branch
from
Open

ui fix for Selected repos #529

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
f2c0260
ui fix for Selected repos
JaYRaNa213 Apr 11, 2026
a4d707e
resolve copilot reviews
JaYRaNa213 Apr 19, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 6 additions & 4 deletions src/popup.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -475,11 +475,13 @@ <h4 class="font-semibold text-xl" data-i18n="noteTitle">Note:</h4>

<!-- Selected Tags Container -->
<div id="selectedRepos"
class="mt-3 min-h-[32px] p-2 border-2 border-gray-200 bg-gray-50 rounded-xl">
<div class="flex flex-wrap gap-1" id="repoTags">
class="mt-3 min-h-[32px] max-h-[120px] overflow-y-auto p-2 border-2 border-gray-200 bg-gray-50 rounded-xl">
Comment thread
JaYRaNa213 marked this conversation as resolved.
Outdated

<div class="flex flex-wrap gap-2" id="repoTags">
<span class="text-xs text-gray-500 select-none" id="repoPlaceholder"
data-i18n="repoPlaceholder">No
repositories selected (all will be included)</span>
data-i18n="repoPlaceholder">
No repositories selected (all will be included)
</span>
</div>
</div>

Expand Down
17 changes: 10 additions & 7 deletions src/scripts/popup.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1552,13 +1552,16 @@ document.addEventListener('DOMContentLoaded', () => {
.map((repoFullName) => {
const repoName = repoFullName.split('/')[1] || repoFullName;
return `
<span class="inline-flex items-center px-2 py-1 bg-blue-100 text-blue-800 text-xs font-medium rounded-full" style="margin:5px;">
${repoName}
<button type="button" class="ml-1 text-blue-600 hover:text-blue-800 remove-repo-btn cursor-pointer" data-repo-name="${repoFullName}">
<i class="fa fa-times"></i>
</button>
</span>
`;
<span class="flex items-center max-w-[120px] px-2 py-1 bg-blue-100 text-blue-800 text-xs font-medium rounded-full m-1">
Comment thread
JaYRaNa213 marked this conversation as resolved.
Outdated
<span class="truncate">${repoName}</span>
<button
type="button"
class="ml-1 text-blue-600 hover:text-blue-800 remove-repo-btn flex-shrink-0 cursor-pointer"
data-repo-name="${repoFullName}">
Comment thread
JaYRaNa213 marked this conversation as resolved.
Outdated
<i class="fa fa-times"></i>
</button>
Comment thread
JaYRaNa213 marked this conversation as resolved.
Outdated
</span>
Comment thread
JaYRaNa213 marked this conversation as resolved.
Outdated
`;
})
.join(' ');
Comment thread
JaYRaNa213 marked this conversation as resolved.
Outdated
Comment thread
JaYRaNa213 marked this conversation as resolved.
Outdated
repoTags.querySelectorAll('.remove-repo-btn').forEach((btn) => {
Expand Down
Loading