Juancavallotti/master#1
Conversation
|
@Frazer, I have reviewed your PR, but I'm not clear on the rationale for removing the Icon from the top bar. Please let me know if this was done by mistake before I merge. Re-credits, I haven't thought about releasing under the same brand name because we don't have the author's permission, but we could keep it released as I did on the repo. You can add a CONTRIBUTORS.md file to the project and add your name and your socials. |
|
If you mean the icon in the MenuBar, I didn't remove it.
I'm looking at your build and mine, and not sure which part you mean.
I thought I suggested some alternative names.
…On Tue, Apr 14, 2026 at 5:51 PM Juan Alberto Lopez Cavallotti < ***@***.***> wrote:
*juancavallotti* left a comment (juancavallotti/ClipMenu#1)
<#1 (comment)>
@Frazer <https://github.com/Frazer>, I have reviewed your merge, but I
don't understand the decision to remove the Icon from the top bar.
Re-credits, I haven't thought about releasing under the same brand name
because we don't have the author's permission, but we could keep it
released as I did on the repo.
You can add a CONTRIBUTORS.md file on the project and add your name and
your socials.
—
Reply to this email directly, view it on GitHub
<#1 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAC3NQALMNFSTADVBHMRUW34V3MKBAVCNFSM6AAAAACXZPYLV2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHM2DENBYGM3DEOBXGQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Fixes paste on selection and polishes popup placement, and app ignore list:
It wasn't pasting when I selected the item, so fixed in these 2 commits - I added UI tests to make sure it worked.
Fix paste-on-selection and add popup flow tests
Protect clipboard history during UI tests
A couple of other functional changes:
Popup is above cursor if triggered on bottom half of screen
Allow apps to be removed from ignore list - They couldn't be selected, so couldn't be removed.
Fix keyboard shortcuts to numbers
Some style changes to fit many folders and snippets in the popup menu
fix folder size for extra clips in popup menu
when items are prefaced with a number, hide number on right