8000 Document lighttable thumbnails in response to: icon to display tags list #18297 by Dave22152 · Pull Request #745 · darktable-org/dtdocs · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Document lighttable thumbnails in response to: icon to display tags list #18297 #745

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 14 commits into
base: master
Choose a base branch
from

Conversation

Dave22152
Copy link
Contributor

Adds description of new tag icon in lighttable thubnails and also adds descriptions of group and history icons which don't seem to be addressed elsewhere

Refers to DT PR 18297


Certain icons are displayed on thumbnails to provide additional information on images, when applicable

Images that are members of a group are denoted by a group icon ![light group icon](./thumbnails/Group-icon-light.png#icon) or ![dark group icon](./thumbnails/Group-icon-black.png#icon), where black identifies the leader image in that group. See [image grouping](https://github.com/Dave22152/dtdocs/blob/master/content/lighttable/digital-asset-management/grouping.md) for details
Copy link
Contributor
@elstoc elstoc Apr 13, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this linking to your own github repo?

@elstoc
Copy link
Contributor
elstoc commented Apr 13, 2025

The images in this PR are a little too small for legible display on the website. You may be able to extract larger images by increasing your DPI and/or font size. Ideally the borders around these images should be as small as possible (again, for better legibility)

@Dave22152 Dave22152 marked this pull request as draft April 14, 2025 12:18
@Dave22152
Copy link
Contributor Author

Thanks @elstoc. I'll fix the link and the icons. How large would you suggest I make them?

< 8000 div class="js-timeline-item js-timeline-progressive-focus-container" data-gid="IC_kwDOESNYaM6m_sWj">
@elstoc
Copy link
Contributor
elstoc commented Apr 14, 2025

I don't know, maybe twice the dimensions of the existing ones? Ideally if you could build the site locally, you could test. There's a sweet spot. Too large and when the website downsizes them the lines are thin and indistinct, too small and they look pixellated

@Dave22152
Copy link
Contributor Author
Dave22152 commented Apr 14, 2025

I corrected that link and enlarged the icons. I'm still working on converting my local repo to HTML, so hopefully this is an improvement

Update: I was able to build the site locally this morning and the changes look ok to me.

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.

2 participants
0