search

Xyphyn / photon

A replacement for lemmy-ui with more features, a better design, and more customizability.
https://phtn.app
GNU Affero General Public License v3.0
344 stars 38 forks source link

Remove duplicate display of link title #289

Closed cloudlessx closed 2 months ago

cloudlessx commented 5 months ago

Is your feature request related to a problem? Please describe. You see the exact same title twice in all link posts.

Example: image

Describe the solution you'd like Show the article title in the header only.

Describe alternatives you've considered Use full-size card instead of thumbnail for links. Like what Voyager does. Perhaps making it an option?

Additional context How Voyager does it: image

BrikoX commented 5 months ago

One is user title another is element from the link. One can be modified by the user, another can't. </p> <p>It's just commonly the same due to generate title feature on Photon and similar functionality in other clients. </p> </div> </div> <div class="comment"> <div class="user"> <a rel="noreferrer nofollow" target="_blank" href="https://github.com/cloudlessx"><img src="https://avatars.githubusercontent.com/u/159134845?v=4" />cloudlessx</a> commented <strong> 5 months ago</strong> </div> <div class="markdown-body"> <p>@BrikoX While I understand the logic behind the design, there is a reason why every other client (and kbin) UI display only one of the titles in the feed. </p> <p>Web UI like Lemmy UI, Voyager, Kbin, Mbin. Apps like Eternity, Raccoon for Lemmy... just to name a few. They all display only the post title by default, some with a button that you can use to expand and see the article title.</p> <p>I am all for letting the users see more information. But in this case the titles become redundant when the user is trying to browse the feed.</p> <p>Edit: just found that Alexandrite behaves similar to Photon.</p> </div> </div> <div class="comment"> <div class="user"> <a rel="noreferrer nofollow" target="_blank" href="https://github.com/Xyphyn"><img src="https://avatars.githubusercontent.com/u/80978739?v=4" />Xyphyn</a> commented <strong> 4 months ago</strong> </div> <div class="markdown-body"> <p>i added this a while ago, but some people complained so i removed it, now we're back to this lol</p> <p>ill just make it an option</p> </div> </div> <div class="comment"> <div class="user"> <a rel="noreferrer nofollow" target="_blank" href="https://github.com/cloudlessx"><img src="https://avatars.githubusercontent.com/u/159134845?v=4" />cloudlessx</a> commented <strong> 4 months ago</strong> </div> <div class="markdown-body"> <p>@Xyphyn How about you make an option that hides the article title if it is the same as the post title? Something like:</p> <p>if(article.title == post.title) { article.title.setVisbility(false); }</p> </div> </div> <div class="page-bar-simple"> </div> <div class="footer"> <ul class="body"> <li>© <script> document.write(new Date().getFullYear()) </script> Githubissues.</li> <li>Githubissues is a development platform for aggregating issues.</li> </ul> </div> <script src="https://cdn.jsdelivr.net/npm/jquery@3.5.1/dist/jquery.min.js"></script> <script src="/githubissues/assets/js.js"></script> <script src="/githubissues/assets/markdown.js"></script> <script src="https://cdn.jsdelivr.net/gh/highlightjs/cdn-release@11.4.0/build/highlight.min.js"></script> <script src="https://cdn.jsdelivr.net/gh/highlightjs/cdn-release@11.4.0/build/languages/go.min.js"></script> <script> hljs.highlightAll(); </script> </body> </html>