akshat: ruaok: looked at the logs, i think the issue here is trying to access element using `getElementById`. react/ts don't like that. you'd need to create `ref` to access it like that. but that is discouraged. afaik, preferred way to do this would be to add a field to the component's state.
using `document.body` is kinda an exception to it because 1) its always there 2) its not created inside react
bitmap
zas: tests still take longer now (18-19 minutes instead of 17) but it is a little better, thanks
lucifer
i have pushed a sample fix for the grid background. you can revert that for going back to full page background color.
bitmap
if you need to access document.body you can normally do it inside componentDidMount() (or useEffect() for functional components)
lucifer
i see. makes sense.
here its the opoosite case, we currently do document.body but want to change to switch to just a div rendered by react.
Lotheric has quit
Lotheric joined the channel
Lotheric has quit
Lotheric joined the channel
Lotheric_ joined the channel
Lotheric has quit
loujine_ joined the channel
Leftmost_ joined the channel
Leftmost has quit
loujine has quit
Leftmost_ is now known as Leftmost
peterhil joined the channel
peterhil has quit
ruaok
mooin!
thanks for the grid fix, preparing to send to test
lucifer: thanks for the fix -- I think the box could use some top/bottom margin, but this is pretty good.
lucifer
agreed.
should i add those? i can probably also throw in some rounded corners if you like.
ruaok
❤️
lucifer
huh. the color wheel seems to broken sometimes but i am unable to reproduce the issue reliably.
ruaok
what do you think of the idea of having the color wheel in the center of the screen when the page loads?
and then when a color is clicked, it moves to its correct position, ideally with a short animation.
lucifer
i think it looks fine. +1 on the animation. the current movements are too abrupt.
ruaok
exactly.
that way we can also adjust the text a bit for each of the two stages.
perhaps just start with "Click on a color below" and the color wheel.
lucifer
same for BP btw, it shifts position from left to right when cover art is clicked abruptly.
thoughts on making BP hidden by default? and only render it when cover art is clicked
ruaok
yerp, would be good to fix.
yes, please to hiding.
lucifer
cool. let's see how far along i can take this.
ruaok
show it when the grid appears and then change the instruction text to "Now click on an album cover to listen to that album".
woo, thanks!
lucifer
👍
ruaok
I have worked out how I want the scripts to stay in sync, but I doubt I'll get them done today. today is a bit fractured for me. (and I still need to prep a state of metabrainz talk)
lucifer
oh nice, the state talks :D
if you can write it up, i'll lend a hand on backend cleanup
(write up what needs to be done to update the scripts)
ruaok
I think it would take longer to explain the tasks at hand than to actually carry it out, but thanks.
what are missing are some tests for the python code, if you feel really heroic. :)
MrClon joined the channel
akshat
moin!
ruaok
moin moin!
akshat
Is there anything more to be done for huesound from my end or lucifer is looking into the orientation?
ruaok
we're nearing the end for this round of changes. not sure if there are any front end tests that we should have, but that is still on the list.
akshat
Oh okay.
Currently I'm back to working on the mb web interface. We can start the huesound tests once we have finalized stuff I would like to help with that whenever we decide to :)
ruaok
I updated the todo items in the PR -- most are mine. I'll try and do some cleanup today, but I won't finish the migration script. that will have to wait until next week, sadly.
ok, sounds good. also need to give you feedback on the new homepage. let me do that this morning.
alastairp
good morning
ruaok
morning!
akshat
Thanks ruaok
alastairp
zas: hi, yes. 1.2T is the size of the AB database now. no way to reduce it, because that's the data that we have
ruaok
oy.
alastairp
however, perhaps we should start looking at our plans for moving data files to disk, so that we can put them in a separate location
ruaok
why is it that I always feel that "that will be a nice problem to have", but then when the time comes I no longer feel that way? thanks past rob. bastard.
alastairp: +1
alastairp
regarding /home, I make periodic db backups into there. I can delete the most recent one that I made
OK, looks like my AB priority list is 1) dumps, 2) new data storage. happy to start working on that next
lucifer do check this out. This will solve all our problems. I'm planning to work some part of the mb web mock with this setup.
yyoung[m]
There're tons of frameworks that'll solve our problems
adhawkins has quit
But considering the current setup of MB codebase, some of them may need great effort to fit into the current system.
akshat
Correct yyoung[m] absolutely agreed! But from my understanding next js seems to be the most intuitive, tested and reliable resource as a solution for our problems
For now I can confirm that CRA won't solve what we're looking for and a nice framework like next js is very handy for us
ruaok
akshat: remember that when you're picking tools, that you need to be really mindful of the rest of the MB team -- they will need to code with it for the foreseeable future. so, they should be your target audience for these things.
yyoung[m]
Right, I just want to remind you not to be too aggressive :) I like bleeding-edge technologies too, I like SPA as well, but the reality is we don't have every API needed for SPA
ruaok doesnt like bleeding edge technologies
ruaok
I like solid working stuff.
akshat
Agreed
yyoung[m]
These tools are cool and easy to use when starting a new project, like your mockup, but for existing system like MB there're tradeoffs to consideer
ruaok
loads and loads of tradeoffs
akshat
Right
yyoung[m]
After doing your research and decision, maybe ask for advice from the MB dev team before directly jumping into it :)
ruaok
akshat: yyoung[m] is exactly on the right track. this can be very sensitive and these tools need careful choosing.
adhawkins joined the channel
akshat
Does make sense. I'll slow down and work as is for now then. Let's wait for more discussion on this stuff. For now I'll continue with the integration of my current work to mb server then
Will finish the mockup based on some feedback received then move ahead
yyoung[m]
Wow have you already started the integration?
akshat
Not yet yyoung[m]
zas
yvanzo: I'm restarting sir-prod on pink
yvanzo: if it doesn't recover you may need to have a look
ruaok do we want to add a Pricing section to the mb homepage?
ruaok
no.
akshat
Cool. I've roved the FAQ section
ruaok
that's MeB's domain.
k
akshat
Okayy
So, Main section, what is mb, brainz projects, understand mb, supporters, download software, footer
I think we can conclude the homepage with these sections
Obviously based on the feedback some improvements to the sections
ruaok
I think supporters could even be smaller -- I still think there are too many sections. but having some big name logos on our home page gives use a lot of cred.
akshat
Yeaah
I'll make the understand mb sections a but smaller
Bit^
Logos as well
Of the supporters
ruaok
yeah.
and the brainzprojects section also feels a bit lengthy.
akshat
Great!
Oh okayy. Maybe let me reduce the font size?
ruaok
I think we mostly need to signpost them from the MB homepage -- a short blurb and the logo is enough.