seeseemelk/cool-retro-term
1
0
Fork 0
Code Issues 449 Pull Requests 12 Actions Packages Projects Releases 8 Wiki Activity

Allow fonts to scale smaller #117

New Issue
Open
opened 2014-09-11 19:20:41 +02:00 by fazjaxton · 13 comments
fazjaxton commented 2014-09-11 19:20:41 +02:00 (Migrated from github.com)
Copy Link

When I maximize the window to 1280 width on my display, I am only getting 58 columns of characters. I have used CTRL+- to zoom out as far as possible. To be more usable, I would like to get at least 80x24 columns of text in an 800x600 window.

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/4326369-allow-fonts-to-scale-smaller?utm_campaign=plugin&utm_content=tracker%2F479407&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F479407&utm_medium=issues&utm_source=github).
When I maximize the window to 1280 width on my display, I am only getting 58 columns of characters. I have used CTRL+- to zoom out as far as possible. To be more usable, I would like to get at least 80x24 columns of text in an 800x600 window. ## <bountysource-plugin> --- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/4326369-allow-fonts-to-scale-smaller?utm_campaign=plugin&utm_content=tracker%2F479407&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F479407&utm_medium=issues&utm_source=github). </bountysource-plugin>
Swordfish90 commented 2014-09-11 19:36:56 +02:00 (Migrated from github.com)
Copy Link

Agreed. At the moment I'm working to another branch which simplify and optimise the underlying architecture. When that is merged this will actually be easier.

Agreed. At the moment I'm working to another branch which simplify and optimise the underlying architecture. When that is merged this will actually be easier.
Swordfish90 commented 2014-09-30 16:49:45 +02:00 (Migrated from github.com)
Copy Link

Hi @fazjaxton I just made a pretty big update. It is now possible to scale the width of the font up to 50% thus doubling the number of columns. Tell me is this solution is enough or if you also need to decrease the minimum scaling value. I tested it on my pc, and the current scaling value of 50% is pretty small.

Hi @fazjaxton I just made a pretty big update. It is now possible to scale the width of the font up to 50% thus doubling the number of columns. Tell me is this solution is enough or if you also need to decrease the minimum scaling value. I tested it on my pc, and the current scaling value of 50% is pretty small.
tomchiverton commented 2015-01-11 10:10:29 +01:00 (Migrated from github.com)
Copy Link

I would like to be able to go even smaller.
On a 1600x900 laptop, even 50% is a very large window to get a day-to-day number of columns in. Why not just allow this slider to go (much) further down rather than trying to make a call on a 'reasonable' limit ?

I would like to be able to go even smaller. On a 1600x900 laptop, even 50% is a very large window to get a day-to-day number of columns in. Why not just allow this slider to go (much) further down rather than trying to make a call on a 'reasonable' limit ?
Swordfish90 commented 2015-01-13 21:53:14 +01:00 (Migrated from github.com)
Copy Link

@tomchiverton The decision was made because scanline and pixel mode look terrible when the number of "virtual pixels" is too close to the number of "real pixels" of the screen. Anyway I agree with you that this solution should not be hardcoded for every users. This just looks one of those things that belong to a hidden config file.

@tomchiverton The decision was made because scanline and pixel mode look terrible when the number of "virtual pixels" is too close to the number of "real pixels" of the screen. Anyway I agree with you that this solution should not be hardcoded for every users. This just looks one of those things that belong to a hidden config file.
tomchiverton commented 2015-01-14 20:33:39 +01:00 (Migrated from github.com)
Copy Link

I don't see the point of a hidden option to enable it. The feedback moving the slider is instant so people will see straight away.
Couldn't it just go from 5% to 150% or something ?

I don't see the point of a hidden option to enable it. The feedback moving the slider is instant so people will see straight away. Couldn't it just go from 5% to 150% or something ?
mengstr commented 2015-01-15 04:05:36 +01:00 (Migrated from github.com)
Copy Link

First I'd just want to say thanks for this cool project.

But +1 to this issue, a 80x25 screen almost fills my MBP, let the scaler go down a bit further. I'd probably be hapy with 25% as a minimum value.

The screenshot below shows the retro-term vs my usial 80x25-sized terminal and even at the small fontsize I use in the regular terminal a lower case 'k' still is 15x16 pixels.

screen shot 2015-01-15 at 10 49 42

First I'd just want to say thanks for this cool project. But +1 to this issue, a 80x25 screen almost fills my MBP, let the scaler go down a bit further. I'd probably be hapy with 25% as a minimum value. The screenshot below shows the retro-term vs my usial 80x25-sized terminal and even at the small fontsize I use in the regular terminal a lower case 'k' still is 15x16 pixels. ![screen shot 2015-01-15 at 10 49 42](https://cloud.githubusercontent.com/assets/325326/5751894/171cc556-9ca6-11e4-8e10-f020465630df.png)
Swordfish90 commented 2015-01-16 11:31:44 +01:00 (Migrated from github.com)
Copy Link

@tomchiverton @SmallRoomLabs How about this: we allow smaller font scaling (we still need to have a reasonable limit like 20% or 25%) with a warning message close the font selector which tells users in scanlines and pixels mode that some graphical issues are expected in the current settings. Do you think that's a reasonable solution?

@tomchiverton @SmallRoomLabs How about this: we allow smaller font scaling (we still need to have a reasonable limit like 20% or 25%) with a warning message close the font selector which tells users in scanlines and pixels mode that some graphical issues are expected in the current settings. Do you think that's a reasonable solution?
mengstr commented 2015-01-16 13:39:17 +01:00 (Migrated from github.com)
Copy Link

Sounds like a plan to me. 👍 Let the users tweak the settings to their hearts desire, but warn them for possible ill effects of the extreme settings.

Sounds like a plan to me. :+1: Let the users tweak the settings to their hearts desire, but warn them for possible ill effects of the extreme settings.
tomchiverton commented 2015-01-16 13:47:16 +01:00 (Migrated from github.com)
Copy Link

@Swordfish90 works for me

@Swordfish90 works for me
yurikhan commented 2015-01-16 15:57:45 +01:00 (Migrated from github.com)
Copy Link

@Swordfish90 Bonus points if this warning message is not a modal message box but just a warning label appearing near the slider, like:

 |--|--.---------|
20%   50%       100%

/!\ At this scale the font may appear suboptimally.
@Swordfish90 Bonus points if this warning message is not a modal message box but just a warning label appearing near the slider, like: ``` |--|--.---------| 20% 50% 100% /!\ At this scale the font may appear suboptimally. ```
Swordfish90 commented 2015-01-20 02:41:20 +01:00 (Migrated from github.com)
Copy Link

This just landed in master. Minimum scaling level is now 25% that should be enough for any use. In the end I improved the rendering of scanlines by using a smart antialiasing technique so there is no need for warning messages.

This just landed in master. Minimum scaling level is now 25% that should be enough for any use. In the end I improved the rendering of scanlines by using a smart antialiasing technique so there is no need for warning messages.
mengstr commented 2015-01-20 04:59:57 +01:00 (Migrated from github.com)
Copy Link

Looks and works fine. I just compiled and installed it without any issues. Thanks.

Looks and works fine. I just compiled and installed it without any issues. Thanks.
tomchiverton commented 2015-01-20 23:09:41 +01:00 (Migrated from github.com)
Copy Link

Builds and works better on Ubuntu.

PPA updates are building.

Builds and works better on Ubuntu. PPA updates are building.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bounty
bug
duplicate
enhancement
help wanted
invalid
priority
high
priority
low
question
wontfix
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
seeseemelk
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: seeseemelk/cool-retro-term#117
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?