Feature #26035
More visually consistent download links
| Status: | Closed | Start date: | ||
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: |  Jean-Philippe Lang | % Done: | 0% | |
| Category: | UI | |||
| Target version: | 3.4.0 | |||
| Resolution: | Fixed |
Description
As a follow-up to #25988 and #25999, this small patch series proposes to move the file download link on previews to a more common place – the contextual area. It also introduces another link wihin the "No preview available" message to make it easier for users to download a file instead of previewing it.
Before:
After:
download_links_after.png (114 KB)
download_links_before.png (108 KB)
0001-Move-download-link-to-contextual-area.patch 
(3.1 KB)
0002-Add-download-link-to-no-preview-message.patch
(5.72 KB)
0003-German-translation-for-download.patch
(945 Bytes)
Related issues
Associated revisions
Move download link to contextual area (#26035).
Patch by Jan Schulz-Hofen.
Add download link to no preview message (#26035).
Patch by Jan Schulz-Hofen.
German translation for download (#26035).
Patch by Jan Schulz-Hofen.
generate i18n keys (#26035)
History
#1
Updated by Go MAEDA over 4 years ago
I think that the improvement is fairly useful but unfortunately the patch cannot be applied to the current trunk (r16559) cleanly. Could you tell me how can I apply them?
$ patch -p1 < 0001-Move-download-link-to-contextual-area.patch patching file app/views/layouts/_file.html.erb Hunk #1 FAILED at 1. 1 out of 1 hunk FAILED -- saving rejects to file app/views/layouts/_file.html.erb.rej patching file app/views/repositories/_link_to_functions.html.erb Reversed (or previously applied) patch detected! Assume -R? [n]
#2
Updated by Jan from Planio www.plan.io over 4 years ago
#3
Updated by Jan from Planio www.plan.io over 4 years ago
- Blocked by Feature #25999: View repository content by default (instead of the history) added
#4
Updated by Go MAEDA over 4 years ago
- Target version changed from Candidate for next minor release to 3.4.0
Jan from Planio www.plan.io wrote:
I am sorry, I failed to mention that #25999 and therefore #25988 are required for this patch to apply cleanly.
Thanks, the patch works fine now.
This small patch makes #25988 and #25999 even better and I think it should be merged along with those issues. Setting target version to 3.4.0.
#5
Updated by Mischa The Evil over 4 years ago
- Related to Feature #25357: "Raw view" of a file in repository added
#6
Updated by Mischa The Evil over 4 years ago
My two cents based on the screenshots:
Jan from Planio www.plan.io wrote:
[...] this small patch series proposes to move the file download link on previews to a more common place – the contextual area.
I'm not sure about this change. The current contextual area does not seem to be the best place for such a link (and an eventual 'raw' link as proposed in #25357). I kinda like the current (and #25999 's) link position (I'm aware of the fact this is really a matter of taste, which is a difficult thing design-wise...).
Jan from Planio www.plan.io wrote:
It also introduces another link wihin the "No preview available" message to make it easier for users to download a file instead of previewing it.
Nice one.
#7
Updated by Jean-Philippe Lang over 4 years ago
- Status changed from New to Closed
- Assignee set to Jean-Philippe Lang
- Resolution set to Fixed
Mischa The Evil wrote:
I'm not sure about this change. The current contextual area does not seem to be the best place for such a link (and an eventual 'raw' link as proposed in #25357). I kinda like the current (and #25999 's) link position (I'm aware of the fact this is really a matter of taste, which is a difficult thing design-wise...).
Yes, I was not sure as well but it seems better than the download link below the tabs after all. Patches are committed, thanks.
#8
Updated by Mischa The Evil over 4 years ago
- Related to Defect #26350: Don't display file download button while on repository directory entries added