turtle/stockpile
1
11
Fork
You've already forked stockpile
8

Show icons in item pages #19

Closed
opened 2023年11月27日 16:10:23 +01:00 by bragefuglseth · 7 comments
Contributor
Copy link

The item pages currently look a little empty. Items are assigned icons upon creation, and it would look nice if these icons were used on the item pages as if they were the icons of AdwStatusPages.

The item pages currently look a little empty. Items are assigned icons upon creation, and it would look nice if these icons were used on the item pages as if they were the icons of AdwStatusPages.
bragefuglseth changed title from (削除) Show icon in item pages (削除ここまで) to Show icons in item pages 2023年11月27日 18:58:44 +01:00

How does this look? Would a larger icon size be better, or is this good?

Skärmbild från 2023年11月27日 22-27-33.png

How does this look? Would a larger icon size be better, or is this good? ![Skärmbild från 2023年11月27日 22-27-33.png](/attachments/f523f64b-1287-419a-b0dc-61af46d95ffb)
Author
Contributor
Copy link

How does it look with AdwStatusPage’s normal icon size (which I think is 128px)?

How does it look with AdwStatusPage’s normal icon size (which I think is 128px)?

The icon appears very large, and especially with icons like the wallet or ticket icons it can appear disorienting, especially with high contrast mode.

Skärmbild från 2023年11月27日 23-14-07.png

I think it looks a tad nicer at 96px:

Skärmbild från 2023年11月27日 23-14-41.png

The icon appears very large, and especially with icons like the wallet or ticket icons it can appear disorienting, especially with high contrast mode. ![Skärmbild från 2023年11月27日 23-14-07.png](/attachments/8a201880-0ee0-404b-b540-7636d4374100) I think it looks a tad nicer at `96px`: ![Skärmbild från 2023年11月27日 23-14-41.png](/attachments/d362cce8-327e-4cbe-a542-d37ff21714eb)
Author
Contributor
Copy link

Can you send a screenshot of the default icon size, but with increased spacing between it and the labels below? I think that could work.

Can you send a screenshot of the default icon size, but with increased spacing between it and the labels below? I think that could work.

Hello, sorry for the delayed response! Here's what the suggestion looks like:

Skärmbild från 2023年12月12日 10-16-32.png

It certainly looks better than with no bottom padding here.

Hello, sorry for the delayed response! Here's what the suggestion looks like: ![Skärmbild från 2023年12月12日 10-16-32.png](/attachments/2fc85aff-c760-4490-8c15-81bef0b75f01) It certainly looks better than with no bottom padding here.
Author
Contributor
Copy link

LGTM!

LGTM!

Yes, I like this look better as well. Closing the issue now since it seems good, please contact me if there are any other issues with the closing of this issue!

Yes, I like this look better as well. Closing the issue now since it seems good, please contact me if there are any other issues with the closing of this issue!
Sign in to join this conversation.
No Branch/Tag specified
main
usage_interval
adw_navigation_split_view
0.5.1
0.5.0
0.4.1
0.4.0
0.3.0
0.2.1
0.2.0
0.1.1
0.1.0
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Reference
turtle/stockpile#19
Reference in a new issue
turtle/stockpile
No description provided.
Delete branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?