Splash screen?

[oldpink]

First, I wanted to say a quick thanks for reviving the previously stagnant XV, still one of the best compact open source image viewers.
The thing that I wanted to make as a request has nothing to do with functionality, but with esthetics, namely the opening splash screen that shows the version as the old 3.10a, even though it is in fact now revised to version 4.1.1.
I'm sure that you would like having that splash reflect that revision change.
Thanks again.

[mdadams]

You are welcome to submit a pull request to fix this.

[oldpink]

I'm embarrassed to say that I have no idea how to submit one.
I also don't have a fixed splash screen.
Sorry
:(

[tanabi]

I may go take a look at this, though I suspect the splash screen is a pre-render and those numbers are baked into it. Hopefully I'm wrong though!

[jn64]

Looks like the version text is pre-rendered (as X bitmap?) and drawn in the splash screen here (along with the revision date, and the copyright statement before it):

xv/src/xvdflt.c

Lines 108 to 118 in ad9e5fe

	i = xv_ver_width + xv_rev_width + 30;
	xbm2pic((byte *) xv_ver_bits, xv_ver_width, xv_ver_height,
	dfltpic, DWIDE, DHIGH, DWIDE/2 - (i/2) + xv_ver_width/2+1, 220+1,252);
	xbm2pic((byte *) xv_rev_bits, xv_rev_width, xv_rev_height,
	dfltpic, DWIDE, DHIGH, DWIDE/2 + (i/2) - xv_rev_width/2+1, 220+1,252);
	xbm2pic((byte *) xv_ver_bits, xv_ver_width, xv_ver_height,
	dfltpic, DWIDE, DHIGH, DWIDE/2 - (i/2) + xv_ver_width/2, 220, 250);
	xbm2pic((byte *) xv_rev_bits, xv_rev_width, xv_rev_height,
	dfltpic, DWIDE, DHIGH, DWIDE/2 + (i/2) - xv_rev_width/2, 220, 250);

A small project for those interested would be to convert the pre-rendered parts to strings like these other parts, which would make it easier to update:

xv/src/xvdflt.c

Lines 120 to 132 in ad9e5fe

	strcpy(str,"Press <right> mouse button for menu.");
	DrawStr2Pic(str, DWIDE/2+1, 241+1, dfltpic, DWIDE, DHIGH, 252);
	DrawStr2Pic(str, DWIDE/2, 241, dfltpic, DWIDE, DHIGH, 250);
	#ifdef REGSTR
	strcpy(str,REGSTR);
	#else
	strcpy(str,"UNREGISTERED COPY: See 'About XV' for registration info.");
	#endif
	DrawStr2Pic(str, DWIDE/2+1, 258+1, dfltpic, DWIDE, DHIGH, 252);
	DrawStr2Pic(str, DWIDE/2, 258, dfltpic, DWIDE, DHIGH, 251);

[oldpink]

That sounds like the logical solution.
I have zero programming skills, but there are probably others who do.
I just wish that your revived repository were getting more notice so that others could lend you a hand.
Thanks

[mdadams]

Hopefully people who are interested in the repository will star it, as this will likely help to increase its visibility. Currently, the repository only has 5 stars.

[tanabi]

I did my part and starred it. I may have time this weekend to do the conversion proposed by jn64.

[galenhuntington]

FWIW I recently added this project to nixpkgs. That may in time bring more attention.

[jn64]

A downside of this is losing the look of the pre-rendered text. I believe it is set in the Tekton font based on architect Frank Ching's handwriting. Somehow it fits with the floating fish and general weirdness of xv. The other text that is drawn from strings uses a 5x9 pixel font with less personality. I still think it's worth changing for ease of maintenance. Just wanted to note for posterity :)

[mdadams]

FWIW I recently added this project to nixpkgs. That may in time bring more attention.

@galenhuntington Thanks. I think that that this will help.

[tanabi]

@mdadams You can close this one out too! Thanks for merging it.

[oldpink]

Wow!
I just compiled/installed it and it looks great.
Thanks a log...closing.