This question is not answered. Helpful answers available: 1. Correct answers available: 1.


Permlink Replies: 22 - Pages: 2 [ Previous | 1 2 ] - Last Post: 23 Nov 23, 21:33 Last Post By: PhilippeCharles Threads: [ Previous | Next ]
JeffTucker

Posts: 7,783
Registered: 31-Jan-2006
Re: Projector - Bug report - Regression related to the slideshow time
Posted: 29 Oct 23, 19:40   in response to: PhilippeCharles in response to: PhilippeCharles
Helpful
  Click to reply to this thread Reply
Attachment p-c.jap (9.6 KB)
Oh, this gets more and more interesting. I can now replicate this error. I don't have a clue about where it's coming from, but Laza will need to try this:

  • Save the attached .jap file on your PC. In fact, put it in an otherwise empty directory.
  • Launch jAlbum.
  • File > Open Project, and open this project file.
  • Take a look at Projector > Lightbox > Main image > Slideshow delay

I have no idea where the 3 seconds is coming from. It's not the skin default, and it's not in the .jap file anywhere.

(Also, my default skin is Neptune, and it doesn't have a variable with the same name, so no cross-contamination possible there.)

ETA: Laza, just a wild hunch, but I recall running into weird problems with setting up JSpinners with non-integer values. For the last few years, I've simply avoided them entirely. A slideshow delay setting certainly doesn't need anything more precise than whole seconds. ;)
JeffTucker

Posts: 7,783
Registered: 31-Jan-2006
Re: Projector - Bug report - Regression related to the slideshow time
Posted: 29 Oct 23, 19:46   in response to: PhilippeCharles in response to: PhilippeCharles
 
  Click to reply to this thread Reply
PhilippeCharles wrote:
I also tried the following....

You can stop experimenting now. As you can see from what I just posted, I've boiled this down to a very simple test case, with no saved default files involved at all. There's certainly nothing you can do about it for now, other than to remember that every time you open a saved project, reset the time before making the album!

I've given Laza a "heads up" about it.
PhilippeCharles

Posts: 113
Registered: 8-Dec-2012
Re: Projector - Bug report - Regression related to the slideshow time
Posted: 4 Nov 23, 16:33   in response to: JeffTucker in response to: JeffTucker
 
  Click to reply to this thread Reply
Hello Jeff,
Thank you so much for your time to have investigated this issue :-).
This looks really like a weird one!
Laza

Posts: 1,665
Registered: 6-Sep-2005
Re: Projector - Bug report - Regression related to the slideshow time
Posted: 5 Nov 23, 08:25   in response to: JeffTucker in response to: JeffTucker
 
  Click to reply to this thread Reply
Yes, thanks Jeff, it's possibly the JSpinner.

I had a lot of trouble even with JComboBoxes with integer values. The only way to make them work is by defining the values as String in GUI, and defining the value as int in the SkinModel. For float values, there's no solution besides offering only a few available options. I hate Java in this regard. They were so anal about type conversions they fucked up even the simplest ones.
JeffTucker

Posts: 7,783
Registered: 31-Jan-2006
Re: Projector - Bug report - Regression related to the slideshow time
Posted: 5 Nov 23, 14:27   in response to: Laza in response to: Laza
 
  Click to reply to this thread Reply
I've spent enough time in JSpinner Purgatory™ to know how to stay out of trouble. For integers, a JSpinner is great - easy for the user, and you never have to error-check the return value. For anything else, they're awful. A JComboBox is great for giving a choice of text values. For anything else, they're awful.

For slideshow timing, I'd just give the user whole seconds. If you want to give them more granularity, give them a spinner in milliseconds, with something like SpinnerNumberModel(4000, 2000, 9000, 500). The 500 interval makes it easier for the user. But if he wants the slideshow timing to be ridiculously precise, he can enter 5637ms! Just remember to do your own /1000f, or the slideshow will never end. ;)
RobM

Posts: 3,729
Registered: 4-Aug-2006
Re: Projector - Bug report - Regression related to the slideshow time
Posted: 5 Nov 23, 15:16   in response to: JeffTucker in response to: JeffTucker
 
  Click to reply to this thread Reply
JeffTucker

Posts: 7,783
Registered: 31-Jan-2006
Re: Projector - Bug report - Regression related to the slideshow time
Posted: 5 Nov 23, 15:53   in response to: RobM in response to: RobM
 
  Click to reply to this thread Reply

That one does seem to behave itself. I have a recollection of running into problems at the max and min values of the range, but the details escape me now. It might have been a problem when using float values, rather than double.

This was several years ago, during one of my lost decades. ;)
PhilippeCharles

Posts: 113
Registered: 8-Dec-2012
Re: Projector - Bug report - Regression related to the slideshow time
Posted: 23 Nov 23, 21:33   in response to: Laza in response to: Laza
 
  Click to reply to this thread Reply
Hello Laza,

Will you fix this issue by changing the slideshow time using an integer input?

One possible issue for such design change is obviously the retro compatibility with existing albums. To me, a selection by seconds is good enough; it has also the advantage to secure the retro compatibility by rounding (or truncing) the already saved value in existing albums.

What is strange anyway, is that this issue should potentially occur in multiple places in different skins? So what is specific to this slide show duration value?

Thank you for your feedback.
Legend
Forum admins
Helpful Answer
Correct Answer

Point your RSS reader here for a feed of the latest messages in all forums