Skip to content

Conversation

@MattHeffron
Copy link
Contributor

FontSampler sample sheet display in column major order.
Added alternative CharacterSets designations.
Updated documentation.

Resolves #2273

Added alternative CharacterSets designations.
Updated documentation.
@MattHeffron MattHeffron self-assigned this Dec 5, 2025
@MattHeffron MattHeffron added the enhancement New feature or request label Dec 5, 2025
@pamoroso
Copy link
Contributor

pamoroso commented Dec 5, 2025

I tested this with the output sent to a window and it works, but FontSampler doesn't seem to handle window repaint and reshape. Is this expected?

@MattHeffron
Copy link
Contributor Author

FontSampler doesn't seem to handle window repaint and reshape. Is this expected?

Yes, this is expected. It's just treating the window as a printer. One and done.

@pamoroso
Copy link
Contributor

pamoroso commented Dec 5, 2025

When I call FontSample with 'DISPLAY as the printer and NIL as the stream type the output goes to the primary output and the sample sheet is not shown in a window:

fontsampler1

To actually show the sample sheet in a window I need to pass T as the printer and 'DISPLAY as the stream type:

fontsampler2

@MattHeffron
Copy link
Contributor Author

@pamoroso I fixed the documentation.

@rmkaplan
Copy link
Contributor

rmkaplan commented Dec 6, 2025

When I call it with
(FontSample '(HELVETICA 10) 0 NIL 'DISPLAY))
I get a blank window.

When I replace the NIL with T, I have gotten misaligned pointer errors and hard crashes

@pamoroso
Copy link
Contributor

pamoroso commented Dec 6, 2025

With commit bed9819 the documentation is now correct and clearer.

Copy link
Contributor

@pamoroso pamoroso left a comment

Choose a reason for hiding this comment

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

It works with no issues in my tests.

@MattHeffron MattHeffron merged commit 4e510f8 into master Dec 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Fontsampler in column major order ?

4 participants