Skip to content

Commit 1bac22c

Browse files
committed
more AI instruction changes
1 parent c2427af commit 1bac22c

1 file changed

Lines changed: 7 additions & 4 deletions

File tree

.github/copilot-instructions.md

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -79,8 +79,14 @@ This is the **IFPA (International Flipper Pinball Association) Companion App** -
7979
- Use FluentUI icons: `FluentIcon.IconName` with `FluentRegular`/`FluentFilled` fonts
8080
- Implement proper layout containers for responsive design
8181

82+
### UI considerations
83+
- Try to create a UI which looks "default" or "native" on the platform. My original inspiration was the iOS Mail app.
84+
- When possible, use Fluent UI icons for consistency
85+
- If an icon doesn't exist in IconFonts.xaml, examine appropriate hex codes here: https://github.com/AathifMahir/MauiIcons/blob/master/src/MauiIcons.Fluent/Icons/FluentIcons.cs
86+
- Avoid using images, which may bloat the app size, whenever possible.
87+
8288
### Error Handling
83-
- Always wrap async operations in try-catch blocks
89+
- Always wrap operations that can throw in try-catch blocks from the ViewModel; never let exceptions crash the app
8490
- Use structured logging with Serilog
8591
- Display user-friendly error messages via `Shell.Current.DisplayAlert`
8692
- Log exceptions with context using `logger.LogError(ex, "message")`
@@ -134,9 +140,6 @@ public partial class SampleViewModel : BaseViewModel
134140
[ObservableProperty]
135141
private string title = "Sample";
136142

137-
[ObservableProperty]
138-
private bool isLoading;
139-
140143
[RelayCommand]
141144
private async Task LoadData()
142145
{

0 commit comments

Comments
 (0)