the true apioforum megathread #184

  Apiology
kit src #1553

yes but what if this thread got deleted for no reason

ubq323 (bureaucrat) src #1554

how and why would that happen

kit src #1557

because it would be funny

Jmzd8 src #1562

i wouldn't think it would be funny

mb src #1563

eight pages.

Jmzd8 src #1564

eight pages.

mb src #1565

yes

ubq323 (bureaucrat) src #1566

today bees are to be deployed

gollark src #1567

As they are on all days.

arcade src #1568

oh no scary

also ubq323 pls code review ;-;

ubq323 (bureaucrat) src #1569

I haven't had much free time recently, I'll look at it when I get a chance though

arcade src #1570

owo thanks! no hurry

skye src #1571

wwhy don't you review some bitches. lol

big brother (bureaucrat) #1572
this post never existed.
arcade src #1573

daym i got owned

ubq323 (bureaucrat) (edited ) src #1576

arcade: this looks good. things:

  • the visited link colour doesn't seem to be quite right (ie, it's not the same as the browser's default). but i think the browser default colour differs across browsers so it's probably not possible to fix this (unless you avoid overriding it entirely on light mode, but i'm not sure if that's a reasonable thing to do?)
  • everything else looks good though, on dark and light themes
  • i notice that you have introduced a !important. if possible it would be good to avoid this but it might not be, i know the css needs some refactoring anyway (and it already has an !important elsewhere anyway).

could you git rebase this into a single commit before i merge it?

(also, in the future, it might be easier to send me patches via git send-email, since it's slightly more convenient to do code review in general if i can point directly to parts of the diff. also, pulling in the commit is then a lot easier, i can just pipe the email to git am. (setting up a proper mailing list for apioforum development is on my to do list as well))

arcade src #1577

nice, will do tmrw as soon as I have time!

ubq323 (bureaucrat) src #1578

nice

Jmzd8 src #1579

good morning (CST)

arcade src #1580

gmugmug (EST)

Jmzd8 src #1581

990 am (boonga)

Jmzd8 src #1582

poduyt dova booshi

kit src #1583

arcade: EST gang

Jmzd8 src #1584

mfw no CST gang :(

arcade src #1585

@ubq323 so basically i can't really get the browser default back ;-; it does change. I eliminated all of the !importants!

Email has been sent with patch, and i'll push the code too

Jmzd8 src #1589

you can push code into g.gh0.pw without any permission?

ubq323 (bureaucrat) src #1590

i think arcade means pushing the code to their mirror on sourcehut https://git.sr.ht/~arcade/apioforum. only i can push code to g.gh0.pw.

Jmzd8 src #1591

oooooh ok, that makes sense

please log in to reply to this thread