Skip to content

Update fragment_media_detail.xml #6290

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

ananas304
Copy link

Description (required)

Fixes #6265

What changes did you make and why?
I worked on displaying missing metadata (author, source, and uploader) on the media detail page. These fields were not previously visible when available in the media object. The XML layout was updated to include TextViews for this information, ensuring that if the data is present, it is shown correctly to users.

Tests performed (required)

Tested ProdDebug on Pixel 6 emulator with API level 30..

Screenshots (for UI changes only)

image


Note: Please ensure that you have read CONTRIBUTING.md if this is your first pull request.

Copy link

✅ Generated APK variants!

@nicolas-raoul
Copy link
Member

Can you mark it as draft? Thank you! :-)

@whym whym marked this pull request as draft April 19, 2025 02:17
@whym
Copy link
Collaborator

whym commented Apr 19, 2025

I went ahead and changed it to draft. Please feel free to change it back when it's ready.

I think for now the part where you fill the new fields with actual data is missing.

@nicolas-raoul
Copy link
Member

Did you get a chance to continue working on this?
The changes to fragment_media_detail.xml look great, but the most important towards fixing #6265 is left. :-)

@ananas304
Copy link
Author

I’ve made good progress with the changes to fragment_media_detail.xml, and I’ve worked through most of the technical setup. However, I’m currently facing an issue related to the app not properly instantiating the app component, as seen in the logcat error. This is preventing the app from running as expected, and I’m trying to resolve it now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: The author, source, and uploaded info is missing from pages
3 participants