[comixed-dev] Re: One last feature for the 0.6.0 branch

  • From: "Darryl L. Pierce" <mcpierce@xxxxxxxxx>
  • To: comixed-dev@xxxxxxxxxxxxx
  • Date: Mon, 8 Jun 2020 08:00:46 -0400

Yeah, I can see how that behavior would happen, how it would be have like a
latency issue. The state wiring for loading a comic is processing requests
sequentially; i.e., you ask for comic 1, it tries to get comic 1 when you
then click or do something that causes it then ask for comic 2, 3 and then
4. The state wiring is still processing the first request so, like a dog
fetching sticks you threw in order, it comes back with the first stick but
you're waiting for the last stick you threw.

I do think that this issue will be fixed when the Pages tab is refactored
to lazily load the set of pages. That's where the time is being spent when
you first go to view a comic's details. I just opened #323 [1] to capture
this requirement for the next release.

[1] https://github.com/comixed/comixed/issues/323

On Mon, Jun 8, 2020 at 7:14 AM Guy Incognito <dmarc-noreply@xxxxxxxxxxxxx>
wrote:

I can’t consistently replicate the issue.  For sure I can get it to occur
– but sometimes it may take one or two steps, other times it’s several.



The first time I tried to replicate this today, it happened with the
second comic I attempted to open.  I found I could “next” to another issue,
but as soon as I returned to the library page and tried to open something
different, I’d get the first comic I had opened.



Then a minute ago, the comic that was being opened was the one I tried to
open previously – it was like CX was a step behind.   That occurred for
about three or four comics, but then CX started returning the same
previously opened comic.





The only thing I can for sure say about this is that the web address is
not changing each time I open a comic (once the issue has been triggered).
If CX freezes on comic 108, the address in the browser will be
http://localhost:7171/comics/108?page=0 – and that never changes, even
when I open comic 109, 200 or 1500.



For now, I’m putting this in the too hard basket.  I’ll need to look at it
over the weekend – probably with a stripped down (comic wise) database.





*From: *Guy Incognito <dmarc-noreply@xxxxxxxxxxxxx>
*Sent: *Monday, 8 June 2020 9:38 AM
*To: *comixed-dev@xxxxxxxxxxxxx
*Subject: *[comixed-dev] Re: One last feature for the 0.6.0 branch



So this is similar to, but distinct from, the loading issue unfortunately.



When this issue is triggered, the CX CMD window is only showing “awaiting
task or notification” messages – when the cache / load issue occurs, you
can see the load and image extraction activity in the CX CMD window.



The cache / loading issue eventually results in the newly selected comic
opening...but in the case described, the newly selected issue is never
loaded.



*From: *Darryl L. Pierce <mcpierce@xxxxxxxxx>
*Sent: *Monday, 8 June 2020 12:20 AM
*To: *comixed-dev@xxxxxxxxxxxxx
*Subject: *[comixed-dev] Re: One last feature for the 0.6.0 branch



Remember, we already have a bug opened against that behavior? But it's the
browser caching the previous images that's the core issue: we've talked
about putting a busy overlay on the screen until the comic cover is loaded,
and having the individual pages lazily loaded so that the browser is
waiting for 20+ images to be loaded for each issue.



On Sun, Jun 7, 2020 at 7:29 AM bareheiny <dmarc-noreply@xxxxxxxxxxxxx>
wrote:

I’ll create a bug for this tomorrow (too tired to work through the steps
now), but I’ve been able to consistently get to the point where no matter
what comic I open, it will always display a previously selected issue.



Meaning after a some library navigation ending in me opening GL #8, no
matter what comic I open after - GL #8 is shown.  Note the comic selected
when getting to this point doesn’t matter...I’ve replicated it with a
number of different comics.



It’s not a case of CX loading the new comic either...CX is waiting for a
task or notification.





On 7/06/2020, at 22:52, bareheiny <bareheiny@xxxxxxxxx> wrote:



Is there a way to delete reading lists?



On 7/06/2020, at 20:51, bareheiny <bareheiny@xxxxxxxxx> wrote:



Right-o then, that works.



As does the new button - could do without the confirmation pop up though :)





On 7/06/2020, at 00:57, Darryl L. Pierce <mcpierce@xxxxxxxxx> wrote:



You create a reading list by right-clicking on the Reading List entry in
the navigation tree. We should probably have one in the toolbar as well?



On Sat, Jun 6, 2020 at 1:50 AM Guy Incognito <dmarc-noreply@xxxxxxxxxxxxx>
wrote:

So I can see the “add to list” action, and selecting it results in a
reading list selection menu....but...I can’t figure out how to actually
create a list.

The selection drop-down box does not seem to allow text entry



*From: *Darryl L. Pierce <mcpierce@xxxxxxxxx>
*Sent: *Saturday, 6 June 2020 9:12 AM
*To: *comixed-dev@xxxxxxxxxxxxx
*Subject: *[comixed-dev] Re: One last feature for the 0.6.0 branch



Release 0.6.0-0.rc6 is now up and ready for testing!



https://github.com/comixed/comixed/releases/tag/v0.6.0-0.rc6



On Fri, Jun 5, 2020 at 4:53 PM Darryl L. Pierce <mcpierce@xxxxxxxxx>
wrote:

I managed to wrap up development on re-implementing adding comics to a
reading list today so I'm going to pull that into the 0.6.0 release and
that's going to be it for new features (and this time I mean it <g>).



So once I create the new release candidate for it (it'll be 0.6.0-0.rc6) I
will only add bug fixes to the that branch as we get it ready for release.
So please grab that later today or in the morning and start testing it and
filing any bugs you find.



Thanks for all the contributions! :D


--

Darryl L. Pierce <mcpierce@xxxxxxxxx>
"Le centre du monde est partout." - Blaise Pascal
"Let's try and find some point of transcendence and leap together." - Gord
Downie



--

Darryl L. Pierce <mcpierce@xxxxxxxxx>
"Le centre du monde est partout." - Blaise Pascal
"Let's try and find some point of transcendence and leap together." - Gord
Downie





--

Darryl L. Pierce <mcpierce@xxxxxxxxx>
"Le centre du monde est partout." - Blaise Pascal
"Let's try and find some point of transcendence and leap together." - Gord
Downie



--

Darryl L. Pierce <mcpierce@xxxxxxxxx>
"Le centre du monde est partout." - Blaise Pascal
"Let's try and find some point of transcendence and leap together." - Gord
Downie







-- 
Darryl L. Pierce <mcpierce@xxxxxxxxx>
"Le centre du monde est partout." - Blaise Pascal
"Let's try and find some point of transcendence and leap together." - Gord
Downie

Other related posts: