Skip to content

Conversation

@spicybiguy
Copy link
Contributor

Updated the selector for the title property for Motherless galleries to be an h2 instead of an h1 to reflect changes on the site

Updated the selector for the title property for Motherless galleries to be an h2 instead of an h1 to reflect changes on the site
@mikf
Copy link
Owner

mikf commented Nov 24, 2025

For both image and video galleries, the gallery title is still inside an <h1> element on my end:

                                                <div class="media-meta-title">
                            <h1>Margaret vs Eva #22</h1>
                        </div>

@mikf
Copy link
Owner

mikf commented Nov 24, 2025

Maybe it would be better/more consistent to extract the title from a gallery's <title> element:

<title>TITLE HERE | MOTHERLESS.COM ™</title>

@spicybiguy
Copy link
Contributor Author

The only H1 tags on a gallery page appear to be for the tab selects (home, images, videos, etc.). For example, this gallery, https://motherless.com/GABE048B, the title is in an H2 tag:

image

@spicybiguy
Copy link
Contributor Author

Ok I may have found the disconnect. The old version of the site still uses an H1 tag but to get to that you ahve to clikc on a link on the page and then it sets a cookie value to load the old version of the site. There is no difference in the URL however so when gallery-dl hits it, it loads the new site version.

@spicybiguy
Copy link
Contributor Author

Screenshot of the old version for comparison
image

@mikf
Copy link
Owner

mikf commented Nov 24, 2025

Your changes affect the title of a single media page (https://motherless.com/GABE048B/174D3B2), not a gallery title (https://motherless.com/GABE048B)

edit: extracting title from the <title> element as mentioned in #8605 (comment) works for both cases.

@spicybiguy
Copy link
Contributor Author

Your changes affect the title of a single media page (https://motherless.com/GABE048B/174D3B2), not a gallery title (https://motherless.com/GABE048B)

edit: extracting title from the <title> element as mentioned in #8605 (comment) works for both cases.

Ah I see I updated the wrong section 🤣 I'll see if I can get it working for the title tag

@mikf mikf merged commit 1115103 into mikf:master Nov 28, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants