Skip to content

Commit

Permalink
Change icons
Browse files Browse the repository at this point in the history
Swap the order of the first two entries: opening PDF should be first.
Introduce icons that better fit the purpose - taken from
https://pictogrammers.com/library/mdi/ and exported as 96x96 PNG
  • Loading branch information
holybiber committed Jul 19, 2024
1 parent 819e9a7 commit b6e6cd8
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 5 deletions.
Binary file added assets/file-document-arrow-right-outline.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added assets/file-document-outline.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added assets/link.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
12 changes: 7 additions & 5 deletions lib/widgets/share_button.dart
Original file line number Diff line number Diff line change
Expand Up @@ -45,16 +45,18 @@ class ShareButton extends ConsumerWidget {
Column(children: [
ListTile(
dense: true,
title: Text(context.l10n.sharePdf),
leading: const Icon(Icons.picture_as_pdf),
title: Text(context.l10n.openPdf),
leading: const ImageIcon(
AssetImage("assets/file-document-outline.png")),
onTap: () {
menuController.close();
},
),
ListTile(
dense: true,
title: Text(context.l10n.openPdf),
leading: const Icon(Icons.picture_as_pdf_outlined),
title: Text(context.l10n.sharePdf),
leading: const ImageIcon(AssetImage(
"assets/file-document-arrow-right-outline.png")),
onTap: () {
menuController.close();
},
Expand All @@ -71,7 +73,7 @@ class ShareButton extends ConsumerWidget {
ListTile(
dense: true,
title: Text(context.l10n.shareLink),
leading: const Icon(Icons.content_copy),
leading: const ImageIcon(AssetImage("assets/link.png")),
onTap: () {
menuController.close();
Share.share(
Expand Down

0 comments on commit b6e6cd8

Please sign in to comment.