Skip to content
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.

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

Font Size is too big in Version 1.5.0 #16

Closed
smaragdus opened this issue May 18, 2020 · 13 comments · Fixed by #17
Closed

Font Size is too big in Version 1.5.0 #16

smaragdus opened this issue May 18, 2020 · 13 comments · Fixed by #17
Assignees
Labels
bug Something isn't working enhancement New feature or request

Comments

@smaragdus
Copy link

@kiichi

For me Copy Cat is an indispensable extension- since I discovered it I have been using it daily, I have clicked the nice cat toolbar button innumerable times. For me it was close to perfect. Until version 1.5.0. I liked the old pop-up window:

Copy Cat version 1.4.3

Copy Cat 1 4 3 - 2020-05-18 - 002

which was very compact. While I find the new pop-up window:

Copy Cat version 1.5.0

Copy Cat 1 5 0 - 2020-05-18 - 002

too large.

I found old CRX files at Crx4Chrome, uninstalled version 1.5.0 and installed version 1.4.3. In attempt to disable automatic update I went to andlmjmbnlaamloflnelcafcnkiplhkc sub-folder, opened manifest.json and edited this line:

"update_url": "https://clients2.google.com/service/update2/crx",

changing it to:

"update_url": "https://localhost",

For the time being the extension has not been updated but I have no idea whether this workaround will work in the long run.

I am aware that someone requested bigger font (#13) so I suppose that the best solution would be if Copy Cat has options where the users would be able to specify which pop-up interface they would like to use:

  • the old one- smaller font, All Tabs on the right;

or:

  • the new one- bigger font, All Tabs on the left;

I definitely prefer the old interface with smaller font and All Tabs on the right. I also do not need the last used command to be remembered as I use almost all copy commands alternately. If options are implemented there might be a check box:

  • Remember last used command

If execution of copy commands via context menu (#12) is implemented there might be anotheer check box in Options:

  • Context menu integration

Regards

@kiichi
Copy link
Owner

kiichi commented May 19, 2020

Hi @smaragdus , thank you for using Copy Cat. I'll make settings screen this week, and how about I'll make this configurable in this way:

  • Font Size or choose different layout to render menu
  • All Tabs left or right

Another person mention the usage like you said here,

I use almost all copy commands alternately

I'll also make copy command configurable.

Context menu integration

Is this mean, turn on / off context menu from the settings? That's possible too.

I also see the glitch on your screenshot, little space under all tabs.

@smaragdus
Copy link
Author

@kiichi

Thank you for your quick response and for considering my suggestion about extension options!

I'll make settings screen this week, and how about I'll make this configurable in this way:

  • Font Size or choose different layout to render menu
  • All Tabs left or right

This will be excellent!

I'll also make copy command configurable.

Nice!

Is this mean, turn on / off context menu from the settings? That's possible too.

Yes, this is exactly what I mean. I would like to be able to access copy commands via context menu but I suppose that some users would prefer just to execute the copy commands via the toolbar button, this would be the case when users have too many extensions and the context menu becomes cluttered. Below I will show examples of 2 other extensions which have option to enable/disable context menu integration:

EditThisCookie 1 5 0 - 2020-05-19 - 003

Reading List 2 4 4 - 2020-05-19 - 003

I suppose that:

  • Show in context menu

check box would be fine.

I suppose that context menu will be implemented with sub-menus for copy commands. I will show you how a similar extension- Copycat, handles context menu integration:

Copycat 2 4 2 - 2020-05-19 - 004

I was about to suggest implementation of context menu myself but I didn't because I was afraid that I would overburden you with too much work.

I am glad that you still have time and motivation to maintain and improve your excellent extension!

Best regards

@smaragdus
Copy link
Author

I saw that context menu integration was implemented in version 1.6.0. I tested it and it works fine.

I suppose that you may consider to shorten a bit the context menu entry:

Copy Cat 1 6 0 - 2020-05-20 - 005

In stead of:

Copy Cat - Quick Copy Title and URL

just:

Copy Cat

I suppose that those who have installed the extension know what it is doing and the description- Quick Copy Title and URL is not that needed.

Thank you for your great work!

Regards

@alejandro5042
Copy link

Yes thank you for working on this extension! I love the attention to polish and quality.

@kiichi
Copy link
Owner

kiichi commented May 23, 2020

Thanks for all feedbacks, I start implementing Settings screen. At least, in the next release 1.7.0 (eta Monday), you can put "Small" font size and "All Tabs" on right again @smaragdus

I'm also looking into where the long text in context menu coming from... it is coming from the manifest of the app, so I shorten it. I'm working on context menu On / Off option now, expecting to release something tomorrow. I'll keep you posted here.

#17

CopyCat_Settings

P.S. For even larger screen folks, I created "Large" Option. @alejandro5042

CopyCat_Settings

@kiichi
Copy link
Owner

kiichi commented May 23, 2020

Lol how come I didn't realized that exact same name and similar plugin exists out there...? he could be my brother in previous life...

https://github.com/BlackGlory/copycat

I might change the name of this plugin if that causes some issue

@kiichi kiichi linked a pull request May 23, 2020 that will close this issue
@kiichi
Copy link
Owner

kiichi commented May 25, 2020

@smaragdus the latest version 1.7.0 is available now. Please go to settings, and set Size to Small and All Tabs on right side. If you need even smaller size, let me know.

@smaragdus
Copy link
Author

@kiichi

First, thank you for your excellent work! I saw that version 1.7.0 was released yesterday and I even tested it but did not have time to reply here. Layout tab in Settings is exactly what I needed and now the pop-up window is exactly to my taste, screen:

Copy Cat 1 7 0 - 2020-05-25 - 002

The context menu is also fine, screen:

Copy Cat 1 7 0 - 2020-05-25 - 010

Lol how come I didn't realized that exact same name and similar plugin exists out there...? he could be my brother in previous life...

The name is not exactly the same- your extension is Copy Cat while the other one is Copycat.

I might change the name of this plugin if that causes some issue

Please don't- the name of your extension and its icon are nice!

I think I found a small issue- the text in Layout tab in Settings does not fit the check-boxes, screen:

Copy Cat 1 7 0 - 2020-05-25 - 005

The lower part of the text looks truncated. Can you confirm this or this appears on my browser only?

Thank you very much for your great work!

Regards

@kiichi
Copy link
Owner

kiichi commented May 26, 2020

OK thank you for screenshots @smaragdus some strange things are happening in Windows.

I'm using PureCSS framework and it should normalize all, but maybe I forgot meta tag's viewport settings... I have extra Windows laptop here, and I'll double check on that. I'll have design updates one more time.

@kiichi
Copy link
Owner

kiichi commented May 26, 2020

That little white line below the all tabs was fixed... that was the checkmark , UTF font, rendered differently in Windows. I just shrink to make sure doesn't push.

image

Font size in the settings page... I blame PureCSS framework, somehow it works correctly when I nested inside the grid. That's odd, form elements are nothing to do with grid though, but I'm just taking their example.

image

5e99825

Both Screenshots were taken in Windows 10 Pro, and I tested zoom in / zoom out too. I'll give another update today. @smaragdus

@smaragdus
Copy link
Author

@kiichi

I am using Cent Browser version 4.2.10.171 (based on Chromium version 80.0.3987.163) on Windows 8 x64. If needed I may check the extension with another Chromium-based browser, I also have Vivaldi.

Regards

@kiichi kiichi changed the title Version 1.5.0 Font Size is too big in Version 1.5.0 May 26, 2020
@kiichi kiichi added enhancement New feature or request bug Something isn't working labels May 26, 2020
@kiichi kiichi self-assigned this May 26, 2020
@kiichi
Copy link
Owner

kiichi commented May 27, 2020

Version 1.7.1 is released. Thank you

@kiichi kiichi closed this as completed May 27, 2020
@smaragdus
Copy link
Author

@kiichi

Excellent work, as usual!

Text in text boxes is no longer truncated, screen:

Copy Cat 1 7 1 - 2020-05-27 - 003

Many thanks!

Best regards

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants