Mahz

Mahz's Dev Journal

@Lord Wraith There's no harm when done honestly. I just think of things from an abuse perspective where it's good practice to set a finite limit on everything.

Also, creating notifications has historically had bad performance (it requires one trip to the database per notification created) but I am working on a fix so that notifications can be created in a bulk insert query: github.com/danneu/guild/pull/242

When this is tested and merged, I'll at least bump up the notification limit from 10 to 20. I can bump it up to 30 if you think it would help.

Mahz's Dev Journal

@BrokenPromise First time seeing that pop up, too. Must be something new Mahz just added in. But it's odd that it shows up in PMs vs. the forum pages, if you click through to the last pages on a section the guild has countless "blank" pages where old content should've been. The statement doesn't show up there at all.

@Carlyle

Yeah, I noticed this too:

- The "page is empty" screen when you click a PM tab with no messages
- The forum reports more pages of topics than there are

I assumed it was just pre-existing behavior, but I guess it's a new bug.

I didn't deliberately change any code that would cause this. I think when I transformed the code to modernize it that some functions return slightly different results than what they used to.

@Exit: Let me look into how I can remove all borders from tables (unless you opt in). The current table behavior doesn't make sense.

@Lord Wraith What's the use-case for creating so many tag notifications?

I do agree about letting you create a @mention notification from an edit though. Though I would probably start off by only sending a notification for edits within 30min or something finite like that.

Mahz's Dev Journal

@BrokenPromise Thanks, good points.

I love your forumception demo. The things people build with our bbcode system are amazing.

Yeah, the horizontal scroll just doesn't work. I'm gonna get rid of it. Thanks again.

>roleplayerguild.com/topics/195384-ss/ooc

omfg the design is so good. How does Exit keep getting away with this?

Edit: Ok, I've removed the horizontal scrolling from tables but it won't go into effect until tomorrow.

Mahz's Dev Journal

The sites working on mobile again so whatever demon you sold your soul to seems to have worked. Good job Mahz

@Dark Cloud

What was the issue before?

Mahz's Dev Journal

Okay, let's have Mahz tackle a real problem now. Something that actually hurts people and causes them anguish.

Given that BBC code works inside of tables, I do not think it is necessary for tables to auto-bold the first row of text. Tables also do not play nice with each other when you put tables inside of tables. The table that is inside a table tends to bold up even if you skip placing it in the first row.

Here's a link for clarity and one users mental breakdown.

@BrokenPromise

Agreed. It was a mistake on my part to make the table so opinionated.

I should remove the styling from the table bbcode.

Though off the top of my head I'm not sure how it should work on small device widths.

The current table implementation uses html/css width=100% and then it adds a horizontal scrollbar if the table content can't fit. I'm not sure if this is useful or not.

Maybe you or a bbcode expert like @Exit can chime in on what they think the right trade-off would be?

What would the ideal table bbcode syntax look like? e.g. maybe [table] by default has width=100% but [table=min] has no width; it's determined by content size.

If anyone has a list of bbcode improvements that are curated to the ones you think are most important, I'd like to read that in general. But in the meantime I could make the table bbcode more general/useful.

How do you do, fellow kids?

Hey

Whoa, I haven't thought of Gaia since I was 17. I'm kinda impressed they are still around but also popular (4000 users online).

Mahz's Dev Journal

I did another sweeping update, so it's possible more random things break like email notifications.

The guild code is much nicer to work with now.

Mahz's Dev Journal

Thanks I think I fixed it.

Mahz's Dev Journal

@Crimson Flame Oh, I see your status update. Looks like the list of recent profile views has broken.

Looking into it.

Mahz's Dev Journal

Database upgraded from postgres 12 to postgres 16 just now

It went better than expected.

Next I will do a big code deployment where I attempt to do some whole scale modernizations. And if that succeeds, I will remove the global nag banner.