mirror of
https://github.com/ArchiveBox/ArchiveBox.git
synced 2026-04-25 17:16:00 +03:00
[GH-ISSUE #1213] Bug: Favicon is not always visible #2256
Labels
No labels
expected: maybe someday
expected: next release
expected: release after next
expected: unlikely unless contributed
good first ticket
help wanted
pull-request
scope: all users
scope: windows users
size: easy
size: hard
size: medium
size: medium
status: backlog
status: blocked
status: done
status: idea-phase
status: needs followup
status: wip
status: wontfix
touches: API/CLI/Spec
touches: configuration
touches: data/schema/architecture
touches: dependencies/packaging
touches: docs
touches: js
touches: views/replayers/html/css
why: correctness
why: functionality
why: performance
why: security
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/ArchiveBox#2256
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @tibequadorian on GitHub (Aug 17, 2023).
Original GitHub issue: https://github.com/ArchiveBox/ArchiveBox/issues/1213
Describe the bug
When archiving any site just using a minimal set of extractors like
--extract title,favicon,readability, the favicon is never visible in the index, and the spinner.gif is shown instead.By adding another extractor to the list, like
pdffor example, the favicon is visible again, as shown below.Steps to reproduce
Screenshots or log output
ArchiveBox version
@pirate commented on GitHub (Aug 17, 2023):
Favicon & title alone are not considered enough for the snapshot to be "archived", which is why it shows the in-progress spinner. We could add readability to the set of things it considers enough for it to qualify as "archived" though.
@tibequadorian commented on GitHub (Aug 17, 2023):
I'm fine with your suggestion as it works out in my case but in general wouldn't it make more sense to show the spinner just as long as the archiving process takes, no matter what extractors are used?
@pirate commented on GitHub (Jan 19, 2024):
Fixed in
19e9c1c2f0, will be out inv0.7.3.