search

jack-ullery / AppAnvil

Graphical user interface for the AppArmor security module (in-progress)
GNU General Public License v3.0
16 stars 12 forks source link

Localize timestamp #78

Closed 54m43lJ closed 7 months ago

54m43lJ commented 7 months ago

This addresses #60 by using the system locale settings to format the timestamp. Will definitely break the unit test for format_timestamp().

54m43lJ commented 7 months ago

Here's what it looks like using different locale settings:

2024-02-20-210557_hyprshot 2024-02-20-210634_hyprshot

jack-ullery commented 7 months ago

Thanks Jiang! This has been super helpful

I made a few minor changes to the code before I merged it. The only thing I needed to fix was the sorting behavior. This required changes to status_column_record.cc, which I didn't want to inflict on you (because that class is full of spaghetti code).

I also removed the offending unit test, as it wasn't super important