Content width missing in 0.12.0-beta.5 version

[corasaniti]

Hi,
"Content width" missing in the settings of version 0.12.0-beta.5. It was very useful to have the graph in full screen mode.
In addition, the time and date of the last result is also missing from the dashboard
Has it been removed or will it be restored?
Thank you

[alexjustesen]

CONTENT_WIDTH should now be passed as an environment variable as of v0.12.0-beta.5: https://docs.speedtest-tracker.dev/getting-started/environment-variables

More broadly I'm going to be moving more feature flag related items off the settings page to the environment level.

[alexjustesen]

In addition, the time and date of the last result is also missing from the dashboard
Has it been removed or will it be restored?

So this was done on purpose for the time being. There's an issue with Filament v3 where the method I was using to display that timestamp would cause charts to disappear: #756

Once it's fixed I'll bring back that timestamp.

[corasaniti]

@alexjustesen
Do you think you can parameterize the font size in the header and in the results table with an environment variable?
Thanks

[alexjustesen]

At the moment there doesn't seem to be a native way to do that in Filament. I'll keep an eye out in case it's added but for now I'm going to say probably not.

https://filamentphp.com/docs/3.x/panels/themes/

[corasaniti]

@alexjustesen
Just one more question. What is the page where the sidebar is created? I would like to add the sidebarCollapsibleOnDesktop flag. Why did you remove it, do you plan to restore it? By collapsing the sidebar I could recover space in my 4:3 screen
Thank you

[alexjustesen]

Oops, forgot about that...

Here are the docs for it, PR welcome and the file is located at app/Providers/Filament/AdminPanelProvider.php.

https://filamentphp.com/docs/3.x/panels/navigation/#collapsible-sidebar-on-desktop

[corasaniti]

@alexjustesen
It doesn't matter, if you restore it in the next release I can wait. I thought you deleted it...

[alexjustesen]

They moved from config based features to methods based so it was their upgrade process that didn't pick it up and I don't use it so didn't notice

[corasaniti]

Thanks, I'll wait for the next release

[corasaniti]

@alexjustesen
Hi Alex I'll leave you a reminder for the next release.
In the dashboard, the arrows indicating the ping trend appear to be backwards. See attach Thanks

[alexjustesen]

So that one is correct, lower ping is better