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

Allow changing display foreground and background colors #16

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

AbeJellinek
Copy link
Member

Some progress on Interlisp/medley#6: no full-color display yet, but this activates previously unused fg and bg options so that the kids can have their dark mode, or sci-fi hacker mode, or whatever suits you.

Usage via run-medley:

  • Sci-fi mode: ./run-medley -fg "#008800" -bg "#111111" -xfull35
  • Dark mode: ./run-medley -fg "#ffffff" -bg "#000000" -xfull35

Pound sign isn't required. Just there for disambiguation.

@AbeJellinek
Copy link
Member Author

See Interlisp/medley#45

@AbeJellinek
Copy link
Member Author

Discuss at meeting before merging

@masinter
Copy link
Member

small size bitmap fonts were hand tuned to be legible on CRT black on white. The development of outline fonts depended on some innovations making the fonts render using grey-scale.

@masinter
Copy link
Member

masinter commented May 8, 2021

I started to update this PR with the idea that it might affect the mouse cursor color, but I didn't understand what the code in the conflicting files were doing.

@nbriggs
Copy link
Collaborator

nbriggs commented May 8, 2021

It has nothing to do with the mouse cursor handling. It was only affecting the X11 idea of what the foreground and background colors are.

Copy link
Member

@masinter masinter left a comment

Choose a reason for hiding this comment

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

would solve #12 #934 if merged -- when I build it
and run ldex -fg EFEFEF -bg 202020
the cursor stays black and doesn't get lost --
with the loss of a little bit of contrast

@nbriggs
Copy link
Collaborator

nbriggs commented Sep 13, 2022

This isn't mergeable in its current state. If we're going to do this, we need to fix things like BlackPixel/WhitePixel which should be ForegroundPixel/BackgroundPixel (and they will default to BlackPixelOfScreen()/WhitePixelOfScreen) and we have to fix the X resource database parsing to include these as all other X11 options are done.

@masinter
Copy link
Member

I was just excited to find a Proof of Concept that solved a frustrating problem of reading my screen. I make fewer typos and get fewer headaches with vifrocolor(T) with medley fonts.

https://us06web.zoom.us/rec/share/u4c7R8g_iIim9VvVPTR8mUPmPK7tsRyBO2PlZ3Pp6TgX0jKFvwppjCsUFVMKd5U.nmzITXVzwNNh1QY6 (Passcode: +%y3G9H!)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants