Labels Milestones

feat(feed): Images in the home feed now take up the full width of the card and displays in lightbox when clicked #125

Merged

trentlarson merged 12 commits from feat/image-feed-view-improvements into master 5 days ago

Conversation 0 Commits 12 Files Changed 6

jsnbuchanan commented 6 days ago

Owner

[0.4.0] - 2025.02.14

Changed

• Images in the home feed now take up the full width of the card.
• Clicking the image previously, would open the image in a new tab. Now, clicking the image opens the image in a lightbox view.

Added

• Clicking an image also now displays an in-app lightbox view of the image.
• The lightbox view includes a download button for the image in mobile view.

## [0.4.0] - 2025.02.14 ### Changed - Images in the home feed now take up the full width of the card. - Clicking the image previously, would open the image in a new tab. Now, clicking the image opens the image in a lightbox view. ### Added - Clicking an image also now displays an in-app lightbox view of the image. - The lightbox view includes a download button for the image in mobile view.

Screenshot 2025-02-14 at 1.07.17 PM.png

132 KiB

Screenshot 2025-02-14 at 1.07.29 PM.png

216 KiB

Screenshot 2025-02-14 at 1.46.26 PM.png

205 KiB

Screenshot 2025-02-14 at 1.46.35 PM.png

254 KiB

jsnbuchanan added 11 commits 6 days ago

53a06be734

deps: updated `nostr-tools` and import

781afd8954

feat(feed): better image formatting, to take up the width of the feed container

77ebd32956

deps: updating the nostr-tools version from 2.7.2 to 2.10.4 to fix a bug I was seeing locally

be1146c7df

fix(feed): long words or urls displayed in feed break the container layout

0511bbc17b

feat(feed): adding image viewer for expanding images found in the feed instead of displaying in a new tab and taking the viewer away from the application

f4c7805266

feat(feed): improving the image viewer, to be more conventional, and also allowing the viewer to download the image on mobile with `...` control

e86cf3c9f2

deps: fixing issues around nostr-tools deps being removed that were necessary

aa0156cdf2

docs(feed): documenting improvements to the image view in the home feed

6fc416d759

chore: cleanup in prep for pull request

130d7ecb01 Merge branch 'feat/image-feed-view-improvements' into fix/adding-nostr-tools-dependency

f4155e557a Merge pull request 'feat(feed): Images in the home feed now take up the full width of the card and displays in lightbox when clicked' (#124) from jsnbuchanan/crowd-funder-for-time-pwa:fix/adding-nostr-tools-dependency into feat/image-feed-view-improvements

Reviewed-on: https://gitea.anomalistdesign.com/trent_larson/crowd-funder-for-time-pwa/pulls/124

jsnbuchanan added 1 commit 6 days ago

f53248cae1

docs(feed): versioning 0.4.0

trentlarson approved these changes 6 days ago

trentlarson merged commit 62def44ebb into master 5 days ago

trentlarson referenced this issue from a commit 5 days ago

Merge pull request 'feat(feed): Images in the home feed now take up the full width of the card and displays in lightbox when clicked' (#125) from feat/image-feed-view-improvements into master

Reviewers

trentlarson approved these changes 6 days ago

The pull request has been merged as 62def44ebb.

Sign in to join this conversation.

Reviewers

Request review

No reviewers

trentlarson

Labels

Apply labels

Clear labels

No items

No Label

Milestone

Set milestone

Clear milestone

No items

No Milestone

Assignees

Assign users

Clear assignees

No Assignees

2 Participants

Notifications

Subscribe

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Write Preview

Loading…

Cancel

Save

There is no content yet.