Looking into post #5 by UltimateRiff which has 1 bug (possibly a regression) about “Change credits for other relationships”, 1 improvement suggestion for “Credited as”, 1 regression about edits submission delay.
bitmap
I'm pretty sure we include a pager in the ws/js output
so it may be possible to improve
reosarevok
Ok, then I can add a ticket
bitmap
yvanzo: the edit submission delay might be MBS-12832
MBS-11811: Make Tracklist editing buttons behave consistently on collapsed mediums
reosarevok
That's what I was thinking of
We intentionally changed the release editor behaviour to *not* run on collapsed mediums, since that could cause edits that are invisible to the editor until they submit
(and might be wrong, or at the very least unchecked)
bitmap
hmm, right
not sure what should happen to the track selections if you select a mediumm and then collapse it
reosarevok
Me neither. I guess we could automatically expand it again if something changes?
That could be a good compromise
bitmap
yeah, maybe
reosarevok
I don't think this is a "delay the release" issue, or even a bug necessarily
But a ticket would be good
yvanzo
bitmap: Is there a ticket for dynamically loading numerous medium in the release relationship editor?
(dynamically/lazily)
reosarevok
"expand all" like in the release editor?
bitmap
there is MBS-7919 which I forgot about, but it's more about collapsing than lazy-loading
I'm not sure if there's one for the dynamic loading aspect
reosarevok
Yeah, that should be set as in beta testing
bitmap
don't see one with search
reosarevok
Hmm, one thing that I saw in that thread re:accessibility, do text readers move by tabbing? Because the fact that tab selects the first autocomplete result might make a mess for blind people if so
bitmap: Would you prefer to edit MBS-7919 to mention lazily loading too, or have a separate ticket for it?
reosarevok
Sorry, not release editor, release index, my bad
bitmap
yvanzo: hmm, I think editing it is fine since they're pretty related features
reosarevok
I would not, actually
I think that ticket is solved by the current editor
(I'd know, it's mine :D )
I would add an improvement ticket to add expand all / collapse all options to both the release editor *and* the relationship editor in one go
yvanzo
The current release relationship editor doesn’t allow collapsing, or am I missing something?
reosarevok
The current *beta* editor I meant :)
It already does what I asked for
The expand all thing with lazy loading is a different feature
jivte has quit
yvanzo
We are only speaking about the release relationship editor here.
reosarevok
(I'd also like that, but it's not an inseparable part of the other, as you can see by the fact the release editor has collapsing and expanding but no option to load all AFAICT)
yvanzo
Or do you mean that lazy loading is implemented the same way for both the release editor and the release relationship editor?
reosarevok
I mean neither has lazy loading now, and we should implement it for both :)
yvanzo
(in beta)
reosarevok
But that it is, IMO, a different feature than the collapsing / expanding option itselfd
*itself
And a different ticket
bitmap
they both have lazy loading?
yvanzo
bitmap: isn’t the beta release relationship editor doing lazy loading when there are too many medium?
reosarevok
I mean, I assume the point is "let the user expand them all and load them in one go", because otherwise, both already do that :)
bitmap
yes
yvanzo
reosarevok: I cannot follow you.
reosarevok
Ok, I'm not sure what the lazy loading issue is then :D
Sorry
I thought this was about having one button to lazy-load all mediums in one go
(Because as far as I can tell, it already lazy-loads each medium on demand)
So, sorry for the confusing talk :D
bitmap
basically it will only load the first n mediums or tracks and the rest are only loaded once you expand them
the 'expand all' button is just a convenience feature for that, but not required for lazy loading
reosarevok
Oh
Wait
Were you asking if we should *document* the feature with a ticket?
I thought there was a bug you were bringing up :D
For documenting that they lazy-load, sure, the same ticket works :D
But I'll add a ticket for expanding all :)
yvanzo
There is a bug indeed, I’m just asking if there is a ticket for this new feature or if we should expand MBS-7919 to include this feature as these are closely related.
I keep writing the shit-tab issue, which might also be appropriate
bitmap
lol
reosarevok
I would talk to aerozol about shortcuts before adding them - I'm sure there's a lot that could be useful if we choose to go that way (e.g., expand/collapse all, lol)
But I'm always a bit worried about those because of sites that implement them in annoying ways which block basic features (sigh, discourse hijacking ctrl+f)
yvanzo
bitmap: what is the threshold for not loading last mediums?
reosarevok
Was it 100 total tracks?
Rings a bell, but
yvanzo
(in the beta release relationship editor)
bitmap
it loads up to 10 mediums or 100 tracks, whichever it hits first