Skip to content

Re-enable the target-resolution + sharpen UI once the feature is ready #41

Description

@prekabreki

The Target Resolution + Sharpen controls are currently hidden (display:none on the wrapping .adv-grid, fetchforge/index.html card 05) because the feature is not ready to ship. Commit: see PR #38.

The elements remain in the DOM deliberately — startDownload() reads #target-res.value and #sharpen.checked for both the download and local flows, so leaving them present makes every job send the off state (target_res=0, sharpen=false) with no JS changes.

This issue is the reminder to unhide, and the place to decide the bar for doing so.

Blocked by

Acceptance criteria

  • All blockers above closed or explicitly waived with a reason.
  • display:none removed from the .adv-grid wrapper and the explanatory comment updated or dropped.
  • A real end-to-end job with sharpen on and a target resolution set produces visibly better output than passthrough (not just "runs without error").

Files

  • fetchforge/index.html — search for Target Resolution + Sharpen are HIDDEN.

Notes

  • Trivial change; the work is the judgment call about whether the blockers are genuinely done.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions