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

"Colour value invalid" when extended colours are selected in mIRC #94

Open
danj2k opened this issue Jan 2, 2021 · 4 comments
Open

"Colour value invalid" when extended colours are selected in mIRC #94

danj2k opened this issue Jan 2, 2021 · 4 comments

Comments

@danj2k
Copy link

danj2k commented Jan 2, 2021

Steps to reproduce:

  1. Open mIRC with dlFilter loaded and running
  2. Choose the "View" menu, then choose "Colors"
  3. Change any item (ideally something that occurs frequently, so the error will show up quickly) to a colour that is not part of the block of 16 at the top of the colour picker, then click OK to close the "Colors" dialog
  4. Observe errors such as [dlFilter] Error: $c: Colour value invalid: 88 (where the number will change depending which colour you picked)

Please could dlFilter be updated to support the full range of colours available in mIRC? This looks to be related to the following lines of code (6239-6240):

  elseif ($1 !isnum 0-15) DLF.Error $ $+ c: Colour value invalid: $1
  elseif (($0 >= 3) && ($2 isnum 0-15)) {
@Sophist-UK
Copy link
Collaborator

@danj2k Thank you for reporting this issue. I agree that the code is incorrect - and I will take a look at it ASAP. I can understand the issue, but I was unable to recreate the error by changing e/g the Topic colour - so as you say it presumably only triggers if the item changed is something like normal text.

@Sophist-UK
Copy link
Collaborator

@danj2k I have made these changes, but I have found that there is another issue that needs to be fixed before we can bump the version and you will get a download. Please feel free to change the two lines in your version from 0-15 to 0-99 in order to fix this temporarily.

@danj2k
Copy link
Author

danj2k commented Jan 2, 2021

I can understand the issue, but I was unable to recreate the error by changing e/g the Topic colour

I apologise for my somewhat imprecise description, I discovered this issue entirely by accident when I was setting up a new "white on black" colour scheme so that mIRC wouldn't blind me all the time (the only white-on-black scheme it has built in is a monochrome one, which didn't suit me), so as I changed a lot of things I'm not sure exactly which one it was that caused the issue.

@Sophist-UK
Copy link
Collaborator

It's not a problem. The cause was obvious enough and easy enough to fix.

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

No branches or pull requests

2 participants