Thanks to visit codestin.com
Credit goes to github.com

Skip to content

tags-on-commits-list - Improve code and visibility - #9940

Merged
fregante merged 9 commits into
mainfrom
tags-list
Aug 6, 2026
Merged

tags-on-commits-list - Improve code and visibility#9940
fregante merged 9 commits into
mainfrom
tags-list

Conversation

@fregante

@fregante fregante commented Aug 5, 2026

Copy link
Copy Markdown
Member

Thank god. I always hated this feature. A few rounds of Claude and maybe we got something cleaner.

Screenshot

Test URLs

https://github.com/refined-github/refined-github/commits/19.5.21.1921

Screenshot

I also added a tag icon like mark-merge-commits-in-list does to make it more visible, even if it kinda looks duplicated now:

Screenshot 1
Before After
Screenshot 3 Screenshot 4

I had tried to match the new tag style but it ended up looking worse I think

@fregante fregante changed the title tags-on-commits-list - Refactor tags-on-commits-list - Improve code and update style Aug 6, 2026
@fregante fregante changed the title tags-on-commits-list - Improve code and update style tags-on-commits-list - Improve code and visibility Aug 6, 2026
@@ -1,166 +1,106 @@
import React from 'dom-chef';
import {$, $$, $$optional} from 'select-dom';
import cache from 'webext-storage-cache/legacy.js';

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No more caching, it made little sense as it took a lot of caching space only to be unused after a single new commit. Unblocks #9875

@fregante
fregante merged commit 07249b7 into main Aug 6, 2026
28 checks passed
@fregante
fregante deleted the tags-list branch August 6, 2026 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

1 participant