#bookbrainz

/

      • CatQuest has quit
      • 2022-07-05 18610, 2022

      • CatQuest joined the channel
      • 2022-07-05 18605, 2022

      • Shubh
        CatQuest: did you face the slow keystroke response on modal or on whole unified form?
      • 2022-07-05 18644, 2022

      • CatQuest
        whole unified form, but it was magnitudes worse in the popup(modal?)
      • 2022-07-05 18659, 2022

      • Shubh
        apart from modal(popup?) thing other components of uf should work fairly smooth, it would be helpful if you can provide me which specific field's performance is worse as compare to entity editor?
      • 2022-07-05 18641, 2022

      • CatQuest
        in the whole form they are all bad, in the mpopdal it was worst in.. typing the annotation
      • 2022-07-05 18644, 2022

      • CatQuest
        iirc
      • 2022-07-05 18602, 2022

      • CatQuest
        all typing in the popmodal was excruciating
      • 2022-07-05 18645, 2022

      • CatQuest
        in the whole form they are equally as bad i mean
      • 2022-07-05 18604, 2022

      • CatQuest
        except edit note it was the least bad
      • 2022-07-05 18608, 2022

      • Shubh
        so like annotation in uf is slower than annotation in entity editor?
      • 2022-07-05 18615, 2022

      • monkey
        I just update the new-creation-form branch from master, and pushed the uf-editor PR again on test.bb
      • 2022-07-05 18637, 2022

      • monkey
        CatQuest: Can you see if the typing aspect is any faster now?
      • 2022-07-05 18644, 2022

      • Shubh
        monkey: do you also think it would be a good idea to have separate row for each work in content tab(like identifier editor)?
      • 2022-07-05 18617, 2022

      • monkey
        Yes! I actually just a minute ago wrote that sentence in the comments I'm compiling :)
      • 2022-07-05 18629, 2022

      • monkey
        Here it is:
      • 2022-07-05 18634, 2022

      • monkey
        > I agree the works should be each on a separate row instead of a multi-select.
      • 2022-07-05 18634, 2022

      • monkey
        One added advantage: in the future we will most likely have a way to specify what page a certain work starts at, and it will be much easier if each is in a separate row.
      • 2022-07-05 18634, 2022

      • monkey
        It would also be helpful if we want to show a bit more information for each work in that tab like author (+ checkbox to copy author rel from AC).
      • 2022-07-05 18659, 2022

      • Shubh
        Yep, also one more thing slow performance on modal might be cause of uf and modal sharing the same root state i.e state update on modal also cause useless update (or render) on form as well.
      • 2022-07-05 18640, 2022

      • monkey
        I figured it would be something like that. How do we prevent that from happening?
      • 2022-07-05 18602, 2022

      • Shubh
        something like freezing form when modal is open?
      • 2022-07-05 18635, 2022

      • monkey
        The issue is that the redux actions that are dispatched affect more than one part of the state?
      • 2022-07-05 18603, 2022

      • monkey
        i.e. they will cause a state change in the modal state as expected, but other parts of the form are also reacting to the same actions?
      • 2022-07-05 18614, 2022

      • Shubh
        not state but more than one component dependent on that state
      • 2022-07-05 18641, 2022

      • monkey
        OK. So more a question of rendering on state change (i.e. a React problem, not a Redux problem), correct?
      • 2022-07-05 18649, 2022

      • Shubh
        hmm something like that
      • 2022-07-05 18611, 2022

      • Shubh
        didn't realize it would be noticeable though, i think it is mainly due to large language dropdown which get render 2-3 times on modal which make this worse
      • 2022-07-05 18655, 2022

      • monkey
        That was why I updated test.bb this morning, hoping it will make it more usable
      • 2022-07-05 18607, 2022

      • Shubh
        but this shouldn't be an issue after async language select, i hope
      • 2022-07-05 18633, 2022

      • monkey
        Speaking of, let me work on that one and see if we can get it merged, then I'll try to update new-creation-from branch and deploy the PR again.
      • 2022-07-05 18652, 2022

      • CatQuest
        monkey: yes i can. pls link me directly to the creation form again pls
      • 2022-07-05 18605, 2022

      • monkey
      • 2022-07-05 18621, 2022

      • monkey
        But I don't think it will be much improved.
      • 2022-07-05 18637, 2022

      • monkey
        We need another bit of time to figure it out, will ask you again when we think it's improved
      • 2022-07-05 18659, 2022

      • CatQuest
        sure!
      • 2022-07-05 18616, 2022

      • CatQuest
        I also have to say this yet again: over engenering "states" and "freezing" and the like is in my humble opinion not the way to go about it. instead doing a round to "de-complexify" code. especially ui/ux code to mak it "dumb" instead of "smart" and instead do "smart" things *after* a person clicks a button or selects or the like. is the right way. look how mb does it. searching and typing has never been an issue there.
      • 2022-07-05 18616, 2022

      • CatQuest
        art of this is elasticsearch I'm sure, we really should switch to solrs soon
      • 2022-07-05 18639, 2022

      • CatQuest
        like an annotation or a review note shoudl jsut be plain old hml text-fields
      • 2022-07-05 18641, 2022

      • monkey
        No, not at all related to the search itself
      • 2022-07-05 18650, 2022

      • CatQuest
        what are "states" exactly?
      • 2022-07-05 18612, 2022

      • CatQuest
        why do we need them?
      • 2022-07-05 18615, 2022

      • CatQuest
        what do they do?
      • 2022-07-05 18625, 2022

      • monkey
        The whole state thing is basically where we store the content of each field in a structured manner, that is then sent to the server as a form
      • 2022-07-05 18637, 2022

      • CatQuest
        ...
      • 2022-07-05 18650, 2022

      • CatQuest
        and if we *don't* have that?
      • 2022-07-05 18608, 2022

      • monkey
        If we don't have that we need to rewrite the entire entity editor :)
      • 2022-07-05 18614, 2022

      • CatQuest
        well...
      • 2022-07-05 18629, 2022

      • CatQuest
        that's what this shoud e. (in my *humble humble* opinion)
      • 2022-07-05 18631, 2022

      • monkey
        what we call state = backbone in the old MB
      • 2022-07-05 18635, 2022

      • CatQuest
        hm
      • 2022-07-05 18641, 2022

      • monkey
        Same same
      • 2022-07-05 18644, 2022

      • monkey
        But different
      • 2022-07-05 18652, 2022

      • CatQuest
        it shoudl be same same
      • 2022-07-05 18658, 2022

      • CatQuest
        not different
      • 2022-07-05 18659, 2022

      • CatQuest
        well
      • 2022-07-05 18625, 2022

      • CatQuest
        what does mb do? it doesnt have these "states" i think?
      • 2022-07-05 18635, 2022

      • monkey
        Yes it does
      • 2022-07-05 18643, 2022

      • CatQuest
        what does it do differently thna bb?
      • 2022-07-05 18645, 2022

      • monkey
        Perhaps not exactly the same tool, but *some* tool
      • 2022-07-05 18657, 2022

      • CatQuest
        that tool seems ot be a hella lot less laggy. tbh
      • 2022-07-05 18615, 2022

      • monkey
        Well, most of the issue is some implementation we don't do correctly, and that's fixable
      • 2022-07-05 18625, 2022

      • CatQuest
        hm
      • 2022-07-05 18659, 2022

      • CatQuest
        it's literally always been a little slow though. like years and years ago. the initiall opening of an entity add/edit page has always taken extra tiem to load
      • 2022-07-05 18631, 2022

      • monkey
        Probably the very same implementation problem that's been there all along…
      • 2022-07-05 18603, 2022

      • CatQuest
        and it means i have to wait before editing like the title or whatever because until the page finishes loading it wont register any imput. and so if i type the title before tht and thne press "guess2 it wont work, evne if i do it after it finishes loading
      • 2022-07-05 18608, 2022

      • CatQuest
        exactly
      • 2022-07-05 18607, 2022

      • CatQuest
        btw. we should have a "guess case" type button on title too
      • 2022-07-05 18613, 2022

      • CatQuest
        all titles, names and so on
      • 2022-07-05 18642, 2022

      • CatQuest
        you can copy the logic direct from mb too. just have two states for now. english and sentence
      • 2022-07-05 18657, 2022

      • CatQuest
        later french/german/whatever can also be muxed in
      • 2022-07-05 18616, 2022

      • monkey
        Shubh: Overall I think the unified form is going very well. Still a few teething issues but nothing that seems unsurmountable.
      • 2022-07-05 18616, 2022

      • monkey
      • 2022-07-05 18619, 2022

      • monkey
        On a separate note Shubh: reading this line, is the MAX_F2_OPTIONS also used as a general max amount of items to show in the dropdown? https://github.com/metabrainz/bookbrainz-site/pul…
      • 2022-07-05 18637, 2022

      • Shubh
        monkey: Yep, random though
      • 2022-07-05 18603, 2022

      • monkey
        OK. As I commented I don't think we should restrict the number of frequently used languages (there are only 21 in the DB) but I can rename and reuse the variable as a max number of items shown. Will make that change now
      • 2022-07-05 18613, 2022

      • monkey
        (if you don't mind)
      • 2022-07-05 18651, 2022

      • Shubh
        sound good to me :)
      • 2022-07-05 18601, 2022

      • ansh
        monkey: while working on the client side with live reload enabled, sometimes I observe that the css is not getting updated when refreshed. So I have to restart the server. Are there any changes that might help?
      • 2022-07-05 18616, 2022

      • monkey
        The css file name doesn't change, so it might be cached by the browser. I usually have my browser console open with the option selected to "Disable cache": https://usercontent.irccloud-cdn.com/file/4RdDkFg…
      • 2022-07-05 18639, 2022

      • monkey
        This is in the browser console settings
      • 2022-07-05 18648, 2022

      • ansh
        Is there any caching done by webpack?
      • 2022-07-05 18610, 2022

      • monkey
        At some level yes
      • 2022-07-05 18635, 2022

      • ansh
        Because hard reloading also does not help in those cases.
      • 2022-07-05 18652, 2022

      • monkey
        Ah, interesting.
      • 2022-07-05 18639, 2022

      • monkey
        You could possibly add a content hash to the filename here: https://github.com/metabrainz/bookbrainz-site/blo…
      • 2022-07-05 18631, 2022

      • monkey
      • 2022-07-05 18659, 2022

      • ansh
        Okay, I'll try it and let you know if I still face that issue
      • 2022-07-05 18628, 2022

      • monkey
        You can check the webpack logs to see if the file is emitted (= written to disk)
      • 2022-07-05 18638, 2022

      • monkey
        If all else fails you can run `npm run build-scss`in the docker container
      • 2022-07-05 18628, 2022

      • ansh
        ok
      • 2022-07-05 18642, 2022

      • ansh
        Also for displaying the reviews, which would look better, showing review like cards (https://critiquebrainz.org/review/) or as tables (https://critiquebrainz.org/release-group/be582610…) ?
      • 2022-07-05 18618, 2022

      • monkey
        Cards certainly at least for the entity page
      • 2022-07-05 18628, 2022

      • monkey
        If we were to have a specific page to show all reviews for an entity ("show all" link sort of thing), conceivably tables might work, but then we're just duplicating critiquebrainz, we might as well directly link to the CB page
      • 2022-07-05 18601, 2022

      • ansh
        I have almost reached something similar to what I proposed and the cb website, I'll show it to you tomorrow.
      • 2022-07-05 18614, 2022

      • monkey
        Nice !
      • 2022-07-05 18631, 2022

      • Shubh
        monkey: is there any known way to append icon at accordion header in bootstrap v4?, i've added one in details tab accordion (https://test.bookbrainz.org/create) but its kinda clunky :(
      • 2022-07-05 18643, 2022

      • monkey
        Nothing I know off the top of my head. Maybe this? https://react-bootstrap-v4.netlify.app/components…
      • 2022-07-05 18618, 2022

      • monkey
        Although I'm not opposed to the CSS solution :)
      • 2022-07-05 18609, 2022

      • Shubh
        for some reason, css solution is slow on transition maybe due to late addition of `.show` class? but not sure any idea?
      • 2022-07-05 18623, 2022

      • monkey
        Yes, that's what I'm guessing too.
      • 2022-07-05 18641, 2022

      • Shubh
        tbh without icon accordion doesn't look like an accordion
      • 2022-07-05 18650, 2022

      • monkey
        Agreed
      • 2022-07-05 18607, 2022

      • Shubh
        good thing, they improve this in v5
      • 2022-07-05 18642, 2022

      • monkey
        I tried adding the transition to the class `collapsing`which is added during the accordion toggle transition, seems to work well:
      • 2022-07-05 18642, 2022

      • monkey
        `div[class~=collapsing]+div[class=card-header] .accordion-arrow`
      • 2022-07-05 18600, 2022

      • monkey
        To be clear, in addition to, not replacing styles for `show`
      • 2022-07-05 18627, 2022

      • monkey
        Hm, it reverses the delay (delay after closing, not after opening) which looks a lot better (reactive?) IMO, but maybe that's not what you're looking for
      • 2022-07-05 18640, 2022

      • Shubh
        this looks smooth, what do you think? https://www.irccloud.com/pastebin/LX5cQy6N/
      • 2022-07-05 18606, 2022

      • monkey
        I looks a bit choppy for me
      • 2022-07-05 18634, 2022

      • Shubh
        hmm, is there way to make this fast but also smooth?
      • 2022-07-05 18630, 2022

      • monkey
        Not that I can think of with css, but I think this works OK:
      • 2022-07-05 18635, 2022

      • monkey
      • 2022-07-05 18647, 2022

      • monkey
        Changed the transition speed a bit
      • 2022-07-05 18637, 2022

      • monkey
        CatQuest: I put the new language-select-that-works-more-like-MB on beta.BB
      • 2022-07-05 18637, 2022

      • monkey
        Could you please have a look and let me know if it feels any faster?
      • 2022-07-05 18650, 2022

      • Shubh
        looking good, lets go with this then :)
      • 2022-07-05 18618, 2022

      • CatQuest
        [18:02] <monkey> If we were to have a specific page to show all reviews I epect ti to wrok like mb: on the edition-group page, reviews for the edition group will show (i'd do as the mb one, right now i actually don't know if it only will show the first one, the higest rated one or any review an RG has randonly)
      • 2022-07-05 18636, 2022

      • CatQuest
        monkey: yes i will test!
      • 2022-07-05 18634, 2022

      • CatQuest
        ok so this is what i'm noticing: typing is much, much faster. but things like selecting after search, language search or entity search (for autor credits etc) as wel las any btutton push, (guess case, add a new author credit, etc) is so. hella. slow
      • 2022-07-05 18623, 2022

      • CatQuest
        typing in isbn is really slow
      • 2022-07-05 18626, 2022

      • CatQuest
        name was fast
      • 2022-07-05 18607, 2022

      • CatQuest
        wow selecting "create new work" after searching and not finding anything takes up so much energy my music player crashed (not kidding)
      • 2022-07-05 18612, 2022

      • CatQuest
        the add work modal is as slow as ever. if anything worse
      • 2022-07-05 18630, 2022

      • CatQuest
        trying ot work in it is hanging my system pretty hard
      • 2022-07-05 18652, 2022

      • monkey
        Ah, test hasn't changed, I was asking to test beta :)
      • 2022-07-05 18622, 2022

      • CatQuest
        ಠ_ಠ
      • 2022-07-05 18637, 2022

      • CatQuest
        sorry
      • 2022-07-05 18655, 2022

      • monkey
        no worries
      • 2022-07-05 18638, 2022

      • CatQuest
        give me a link to the thing on beta then. i'm ultra lazy and should go and make dinner but i cna test things just before that for sure
      • 2022-07-05 18600, 2022

      • CatQuest
        you got mad about the ultralazy
      • 2022-07-05 18622, 2022

      • CatQuest
        anyway let me see
      • 2022-07-05 18618, 2022

      • CatQuest
        monkey: you should put things on test before beta you know... if things on beta contradict new fixes, then perhaps we need *another* test server. gsoc.bookbrainz.org? whatever
      • 2022-07-05 18612, 2022

      • CatQuest
        anyway the language selection now is... snappy! :o in comparason
      • 2022-07-05 18618, 2022

      • CatQuest
        well done
      • 2022-07-05 18603, 2022

      • CatQuest
        anyway i cna finally add quite a few books that's been waiting. so i thank you a lot for this