You cannot select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
59d060622d
Follow #22719 ### Major changes 1. `ServerError` doesn't do format, so remove the `%s` 2. Simplify `RenderBranchFeed` (slightly) 3. Remove unused `BranchFeedRSS` 4. Make `feed.RenderBranchFeed` respect `EnableFeed` config 5. Make `RepoBranchTagSelector.vue` respect `EnableFeed` setting, otherwise there is always RSS icon 6. The `(branchURLPrefix + item.url).replace('src', 'rss')` doesn't seem right for all cases, for example, the string `src` could appear in `branchURLPrefix`, so we need a separate `rssURLPrefix` 7. The `<a>` in Vue menu needs `@click.stop`, otherwise the menu itself would be triggered at the same time 8. Change `<a><button></button></a>` to `<a role=button>` 9. Use `{{PathEscapeSegments .TreePath}}` instead of `{{range $i, $v := .TreeNames}}/{{$v}}{{end}}` Screenshot of changed parts: <details>    </details> ### Other thoughts Should we remove the RSS icon from the branch dropdown list? It seems too complex for a list UI, and users already have the chance to get the RSS feed URL from "branches" page. --------- Co-authored-by: 6543 <6543@obermui.de> Co-authored-by: silverwind <me@silverwind.io> |
1 year ago | |
|---|---|---|
| .. | ||
| admin | 1 year ago | |
| auth | 1 year ago | |
| devtest | 1 year ago | |
| events | 2 years ago | |
| explore | 1 year ago | |
| feed | 1 year ago | |
| healthcheck | 1 year ago | |
| misc | 1 year ago | |
| org | 1 year ago | |
| repo | 1 year ago | |
| shared | 1 year ago | |
| user | 1 year ago | |
| auth.go | 2 years ago | |
| auth_windows.go | 2 years ago | |
| base.go | 1 year ago | |
| goget.go | 2 years ago | |
| home.go | 1 year ago | |
| metrics.go | 2 years ago | |
| nodeinfo.go | 2 years ago | |
| swagger_json.go | 1 year ago | |
| web.go | 1 year ago | |
| webfinger.go | 1 year ago | |
