this post was submitted on 24 Feb 2025

35 points (100.0% liked)

Summit

814 readers

55 users here now

Community to discuss Summit, a Lemmy reader for Android.

App (Play Store): https://play.google.com/store/apps/details?id=com.idunnololz.summit

APK: https://github.com/idunnololz/summit-for-lemmy/releases

Support the app

Website: https://summit.idunnololz.com/

founded 2 years ago

MODERATORS

idunnololz

[Release][v1.55.5] Fulfilling more user requests (self.summit)

submitted 1 week ago* (last edited 5 days ago) by idunnololz to c/summit

31 comments fedilink hide all child comments

This is a pretty big release with a large number of UI changes.

This release adds a lot of user requested features to the app, the biggest one being the post feed header. The post feed header adds some information about the feed to the top of the page. The post feed header is disabled by default and will need to be enabled.

This release also updated the designs of a lot of key UI elements. To be honest I've spent so much time working on the UI that I can no longer tell if it's better or worse than before. User feedback on the new UI is greatly appreciated.

Full changelog

Add a setting to disable auto-linking IP addresses. Note that auto-linking IP addresses comes free with Android so enabling this feature will actually cause the app to strip them which is a bit slower.
Add a new layout: full with cards. This layout is the full layout but each post is contained within a card instead of full bleed.
Add video caching.
Add new post feed header. This is disabled by default and can be configured in Settings > Posts feed > Use posts feed header.
Change the UI for the post feed toolbar.
Change video volume logic to be smarter.
Change the left side panel to have subscribed communities sorted by name.
Change icons around the app from circles to rounded squares.
Change some icon designs.
Fixed a bug where sometimes expanding the context in the message screen would cause weird behavior.
Fix locales being mixed together. (Attempt number 2)
Fix a bug where inline video players are not destroyed properly leading to wasted resources.
Fix a bug where cache directories are not cleaned up properly.
Fix a bug where vote colors are not updated immediately in some places.
Fix some minor UI bugs in the post feed.
Show an error icon if an image fails to load.
Adjust left side panel UI.
Adjust compact layout, card1 layout, card2 layout, full layout, full with cards layout.
Adjust comment header layout.
Fix a bug where when a post with an image is expanded while using the list with cards layout the post would have too much padding.
Fix a bug where a video in full screen would continue to play even if the screen is closed.
Fix crash when all subscribed feed is open when feed headers are enabled.
Initial color picker changes. Changes are bit rough around the edges, will polish in the next release.
Fix bug where feed headers are blank for certain types of feeds.

Update

User reported some UI issues with certain layouts. Will fix and rerelease as v1.55.1

Fix a bug where some layouts would show an empty image for text only posts.

Update 2

Some more user reported bugs. Fixing and re-releasing at v1.55.2

Fix a bug where some preferences were being ignored or was broken.
Fix a bug where disabling community icons would require the user to leave the app and come back for the change to be applied.
Fix a bug where the layout "List with cards" option was shown twice.
Minor UI touch ups.

Update 3

I am not releasing this version to the masses just yet because of all of the UI and internal/under-the-hood changes. I'm going to give all the changes some time to sit first.

I don't have an ETA for the release yet but it should definitely be fully live by the weekend.

Thank you to all the early adopters for helping test the changes.

Update 4

I think I'm done with letting things sit and I have a much better idea of what I want the UI to be like. I will do a hopefully final minor release. Changes are:

Show an error icon if an image fails to load.
Adjust left side panel UI.
Adjust compact layout, card1 layout, card2 layout, full layout, full with cards layout.
Adjust comment header layout.
Fix a bug where when a post with an image is expanded while using the list with cards layout the post would have too much padding.
Fix a bug where a video in full screen would continue to play even if the screen is closed.

Update 5

Found more bugs. Addressing before release to all.

Fix a bug where some inline images are really small.
Hide the "All subscribed" feed if user is only signed into one account.
Make the description in the feed toolbar expandable.

Update 6

Rollout complete.

Update 7

User reported a pretty bad crash when all subscribed feed is selected. Will issue an emergency release for this (v1.55.5). I was in the middle of working on some substantial changes so this release will also include those:

Fix crash when all subscribed feed is open when feed headers are enabled.
Initial color picker changes. Changes are bit rough around the edges, will polish in the next release.
Fix bug where feed headers are blank for certain types of feeds.

all 32 comments

sorted by: hot top controversial new old

[–] [email protected] 2 points 3 days ago (2 children)

There seems to be a memory leak in the most recent version of the app, causing severe slowdowns after extended usage of the app (apparent after approximately 10 pages of use, opening every post detail screen while going through the pages)

[–] idunnololz 1 points 1 day ago (1 children)

Can you please check if this issue is fixed in the latest release? Thanks in advance!

[–] [email protected] 1 points 1 day ago

I believe it is, I've not had any issues since. I'll let you know if that changes.

[–] idunnololz 2 points 3 days ago

Thanks for reporting. Will try to reproduce on my end.

[–] [email protected] 2 points 4 days ago* (last edited 4 days ago) (1 children)

Hello,

First, thanks for the great app!

I just want to report a small bug:

When tagging a user, if I enter a text and then change the colour, then the text is erased. Note: it happends wen changing thw two colour type

[–] idunnololz 2 points 3 days ago (1 children)

Thanks for reporting the issue. I will fix this in the next release.

[–] [email protected] 2 points 3 days ago

👍 Have a nice day

[–] Glitterkoe 2 points 6 days ago* (last edited 6 days ago) (1 children)

Hi there! I had to reinstall the app because the app crashes when the homepage was set to "All subscribed" before the update. "Subscribed" and "All subscribed" are also missing from the left menu, but can be accessed from the top header menu after clicking the feed name. They work, but make the app crash on restart. I reset All subscribed as my default home feed in settings, but that reintroduced crashing on startup.

I pretty much only use the subscribed feed, so this is a huge bummer!

Reproduction update: cleared app storage, restarted app, logged in, set All subscribed as home feed, restarted app --> crash

Luckily, regular "Subscribed" seems to work fine, but is a bit hidden since the top bar community name menu has different options as opposed to the menu when swiping from the left.

[–] idunnololz 2 points 5 days ago (1 children)

Thank you for reporting this. I'm going to try to rush to get a fix out ASAP.

[–] Glitterkoe 3 points 5 days ago (1 children)

Awesome! Thank you for your superb app and support!

[–] idunnololz 1 points 5 days ago

Should be fixed in the latest release. Let me know if you notice any other issues.

[–] anon6789 5 points 1 week ago (1 children)

List with cards seems to be on the options list twice.

I feel mixed about the new UI.

I use List and now it feels pretty heavily padded. Compact is way too tight, but I feel old List let me see 2.more posts per screen. There's just a lot of dead space now. If that thumbnail were tucked up in the enjoy space above it, I think that's what it was like before and it seemed better to me.

The padding on the subscribed list you drag in from the left is somewhat the same in that now I have to scroll considerably more. It is nice seeing the community name and the c/whatever since it isn't always the same, and it's probably easier to click the right thing. I didn't have an issue before, and I think the new way looks nicer, but I like it less to use, so I'm split on this one.

I live you're still trying stuff though. Last week I would have said the app is perfect as-is, but I'm willing to play around with experiments to see if anything can still be improved. I can't say enough what a great job you've done on this app though!

[–] idunnololz 2 points 1 week ago* (last edited 1 week ago) (1 children)

List with cards seems to be on the options list twice.

Not sure how that bug sneaked in but I will fix it. Thanks for reporting!

I use List and now it feels pretty heavily padded. Compact is way too tight, but I feel old List let me see 2.more posts per screen. There’s just a lot of dead space now. If that thumbnail were tucked up in the enjoy space above it, I think that’s what it was like before and it seemed better to me.

I actually didn't change the List layout with the exception of adding a hair of more padding between the header and the title which I assume should not be noticable. Were you using the community icons before? They were added a long time ago. Maybe something screwed up the community icon setting but you can disable it in the settings to reclaim some space. It's in Settings > Post feed > Post feed appearance > Use community icons (might require you to back out of the app and come back, just saw this bug now and will fix).

The padding on the subscribed list you drag in from the left is somewhat the same in that now I have to scroll considerably more. It is nice seeing the community name and the c/whatever since it isn’t always the same, and it’s probably easier to click the right thing. I didn’t have an issue before, and I think the new way looks nicer, but I like it less to use, so I’m split on this one.

Very fair. There is a lot to balance between UI design, usability and how much information is shown. I will probably continue to polish the UI in the next couple of releases and will keep your feedback in mind.

[–] anon6789 3 points 1 week ago

I had the community icons on as I noticed they changed from circles to rounded squares.

I do see that switching them off moves the OP's name up to the top line with the post title and that shrinks it a noticeable bit. I like the icons though, they make the feed look more lively.

Perhaps it's just my imagination then, noticing that there were a number of changes and just thinking everything is different.

[–] [email protected] 4 points 1 week ago (1 children)

Looks like any post I reply to now gives the warning about replying to an old post/comment, no matter the actual age of the post/comment.

[–] anon6789 2 points 1 week ago (1 children)

Same here.

[–] idunnololz 3 points 1 week ago

Thanks for reporting. Will look into the issue.

[–] [email protected] 4 points 1 week ago

Love u.

[–] ByteMe 3 points 1 week ago

Damn, these are a lot of changes, can't wait to try them

[–] [email protected] 2 points 1 week ago* (last edited 1 week ago) (1 children)

There seems to be a weird thingy. I've selected the card 2 post feed option (I always use this) and after the update I see image holders (rectangles) in every post that don't have images at all..

[–] idunnololz 4 points 1 week ago (1 children)

Thanks for reporting the issue. I am able to reproduce the problem and will release a fix soon. The only unfortunate things is right after I made the release I did a full project clean up which might break more things. Fingers crossed :D.

[–] [email protected] 2 points 1 week ago

Fixed with the latest 1.55.1. 👍

[–] ByteMe 2 points 1 week ago (1 children)

Hey, I love the post headers but maybe they are not needed in subscribed/all feeds?

[–] idunnololz 1 points 1 week ago (1 children)

Oh i actually didnt test the headers with the subscibed feed. I'll make a note to fix it for the subscribed feed since right now it's just a lot of blank space. As for conditionally showing them, thats a bit more complex. I can make a setting to only show them for individual communities. Ill add that to the roadmap.

[–] ByteMe 2 points 1 week ago (1 children)

Maybe not showing them at all on these 2 feeds?

[–] idunnololz 2 points 6 days ago (1 children)

I think for consistency it's better to show them by default. But I will add a setting to change it once I get to it.

[–] ByteMe 1 points 6 days ago (1 children)

But I don't think it's any useful

[–] idunnololz 2 points 5 days ago (2 children)

there is actually some use to it which is it pushes content down. On devices with large screens it cna be difficult to access items at the top of thr screen with one hand. A header pushes content lower making it easier to use the app with one hand.

[–] ByteMe 2 points 4 days ago (1 children)

I think it will look better if you remove the banner and the icon which are blank now. Leave the space though

[–] idunnololz 1 points 3 days ago

These should not be blank. They should be filled by the feed icon and the banner should be using a default. If they are blank then that is a bug.

How it's supposed to look:

[–] ByteMe 2 points 5 days ago

Oh, right right.