Compare commits
7 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| 74a5426755 | |||
| 851791e02f | |||
| 8b6950f88b | |||
| 8c7e74286b | |||
| 77f661fa2c | |||
| 216d9beac2 | |||
| ca49cdbbf3 |
@@ -38,6 +38,7 @@ func (h *Handler) RegisterRoutes(mux *http.ServeMux) {
|
|||||||
|
|
||||||
// Issues.
|
// Issues.
|
||||||
mux.HandleFunc("GET /issues", h.ListIssues)
|
mux.HandleFunc("GET /issues", h.ListIssues)
|
||||||
|
mux.HandleFunc("GET /issues/new", h.NewIssue)
|
||||||
mux.HandleFunc("POST /issues", h.CreateIssue)
|
mux.HandleFunc("POST /issues", h.CreateIssue)
|
||||||
mux.HandleFunc("POST /issues/{owner}/{repo}/{index}/labels", h.ApplyLabels)
|
mux.HandleFunc("POST /issues/{owner}/{repo}/{index}/labels", h.ApplyLabels)
|
||||||
mux.HandleFunc("POST /issues/{owner}/{repo}/{index}/close", h.CloseIssue)
|
mux.HandleFunc("POST /issues/{owner}/{repo}/{index}/close", h.CloseIssue)
|
||||||
@@ -114,6 +115,10 @@ var basePage = template.Must(template.New("base").Parse(`<!DOCTYPE html>
|
|||||||
<script src="/static/htmx.min.js"></script>
|
<script src="/static/htmx.min.js"></script>
|
||||||
</head>
|
</head>
|
||||||
<body>
|
<body>
|
||||||
|
<header class="top-bar">
|
||||||
|
<span class="top-bar-title">Gitea Mobile</span>
|
||||||
|
<button class="refresh-btn" hx-get="" hx-target="#main-content" hx-swap="innerHTML" aria-label="Refresh">↻</button>
|
||||||
|
</header>
|
||||||
<div class="content" id="main-content">
|
<div class="content" id="main-content">
|
||||||
{{.Content}}
|
{{.Content}}
|
||||||
</div>
|
</div>
|
||||||
@@ -469,6 +474,41 @@ func (h *Handler) PullDetail(w http.ResponseWriter, r *http.Request) {
|
|||||||
renderPage(w, r, fmt.Sprintf("PR #%d", index), "pulls", buf.String())
|
renderPage(w, r, fmt.Sprintf("PR #%d", index), "pulls", buf.String())
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// NewIssue handles GET /issues/new — renders the create-issue form.
|
||||||
|
func (h *Handler) NewIssue(w http.ResponseWriter, r *http.Request) {
|
||||||
|
token := getToken(r)
|
||||||
|
|
||||||
|
repos, err := h.Client.ListOrgsAndRepos(r.Context(), token)
|
||||||
|
if err != nil {
|
||||||
|
slog.Error("failed to list repos for new issue form", "error", err)
|
||||||
|
renderPage(w, r, "New Issue", "issues",
|
||||||
|
`<h1>New Issue</h1><p class="empty">Error loading repositories.</p>`)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
tmpl, err := template.ParseFiles("internal/templates/create_issue.html")
|
||||||
|
if err != nil {
|
||||||
|
slog.Error("failed to parse create_issue template", "error", err)
|
||||||
|
http.Error(w, "template error", http.StatusInternalServerError)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
type templateData struct {
|
||||||
|
Repos map[string][]giteaclient.Repo
|
||||||
|
}
|
||||||
|
|
||||||
|
data := templateData{Repos: repos}
|
||||||
|
|
||||||
|
var buf strings.Builder
|
||||||
|
if err := tmpl.ExecuteTemplate(&buf, "content", data); err != nil {
|
||||||
|
slog.Error("failed to execute create_issue template", "error", err)
|
||||||
|
http.Error(w, "template error", http.StatusInternalServerError)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
renderPage(w, r, "New Issue", "issues", buf.String())
|
||||||
|
}
|
||||||
|
|
||||||
// CreateIssue handles POST /issues.
|
// CreateIssue handles POST /issues.
|
||||||
func (h *Handler) CreateIssue(w http.ResponseWriter, r *http.Request) {
|
func (h *Handler) CreateIssue(w http.ResponseWriter, r *http.Request) {
|
||||||
token := getToken(r)
|
token := getToken(r)
|
||||||
@@ -483,6 +523,12 @@ func (h *Handler) CreateIssue(w http.ResponseWriter, r *http.Request) {
|
|||||||
body := r.FormValue("body")
|
body := r.FormValue("body")
|
||||||
|
|
||||||
if owner == "" || repo == "" || title == "" {
|
if owner == "" || repo == "" || title == "" {
|
||||||
|
if isHTMX(r) {
|
||||||
|
w.Header().Set("Content-Type", "text/html; charset=utf-8")
|
||||||
|
w.WriteHeader(http.StatusBadRequest)
|
||||||
|
fmt.Fprint(w, `<span class="empty">owner, repo, and title are required</span>`)
|
||||||
|
return
|
||||||
|
}
|
||||||
http.Error(w, "owner, repo, and title are required", http.StatusBadRequest)
|
http.Error(w, "owner, repo, and title are required", http.StatusBadRequest)
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
@@ -490,6 +536,12 @@ func (h *Handler) CreateIssue(w http.ResponseWriter, r *http.Request) {
|
|||||||
issue, err := h.Client.CreateIssue(r.Context(), token, owner, repo, title, body, nil)
|
issue, err := h.Client.CreateIssue(r.Context(), token, owner, repo, title, body, nil)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
slog.Error("failed to create issue", "error", err)
|
slog.Error("failed to create issue", "error", err)
|
||||||
|
if isHTMX(r) {
|
||||||
|
w.Header().Set("Content-Type", "text/html; charset=utf-8")
|
||||||
|
w.WriteHeader(http.StatusInternalServerError)
|
||||||
|
fmt.Fprint(w, `<span class="empty">Failed to create issue. Please try again.</span>`)
|
||||||
|
return
|
||||||
|
}
|
||||||
http.Error(w, "failed to create issue", http.StatusInternalServerError)
|
http.Error(w, "failed to create issue", http.StatusInternalServerError)
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -1,7 +1,9 @@
|
|||||||
{{define "content"}}
|
{{define "content"}}
|
||||||
<h1>Create Issue</h1>
|
<h1>Create Issue</h1>
|
||||||
|
|
||||||
<form hx-post="/issues" hx-swap="innerHTML" hx-target="#main-content">
|
<div id="form-error" class="empty" style="display:none;"></div>
|
||||||
|
|
||||||
|
<form id="create-issue-form" hx-post="/issues" hx-swap="innerHTML" hx-target="#main-content">
|
||||||
<div class="form-group">
|
<div class="form-group">
|
||||||
<label for="repo-select">Repository</label>
|
<label for="repo-select">Repository</label>
|
||||||
<select id="repo-select" name="owner_repo" required>
|
<select id="repo-select" name="owner_repo" required>
|
||||||
@@ -32,11 +34,43 @@
|
|||||||
</form>
|
</form>
|
||||||
|
|
||||||
<script>
|
<script>
|
||||||
// Split owner/repo from select into hidden fields.
|
(function() {
|
||||||
document.getElementById('repo-select').addEventListener('change', function() {
|
var repoSelect = document.getElementById('repo-select');
|
||||||
var parts = this.value.split('/');
|
var ownerInput = document.getElementById('owner-input');
|
||||||
document.getElementById('owner-input').value = parts[0] || '';
|
var repoInput = document.getElementById('repo-input');
|
||||||
document.getElementById('repo-input').value = parts[1] || '';
|
var formError = document.getElementById('form-error');
|
||||||
});
|
|
||||||
|
function splitOwnerRepo() {
|
||||||
|
var val = repoSelect.value;
|
||||||
|
if (val) {
|
||||||
|
var parts = val.split('/');
|
||||||
|
ownerInput.value = parts[0] || '';
|
||||||
|
repoInput.value = parts[1] || '';
|
||||||
|
} else {
|
||||||
|
ownerInput.value = '';
|
||||||
|
repoInput.value = '';
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
repoSelect.addEventListener('change', splitOwnerRepo);
|
||||||
|
|
||||||
|
// Validate before HTMX submit.
|
||||||
|
document.getElementById('create-issue-form').addEventListener('htmx:configRequest', function(evt) {
|
||||||
|
splitOwnerRepo();
|
||||||
|
if (!ownerInput.value || !repoInput.value) {
|
||||||
|
evt.preventDefault();
|
||||||
|
formError.textContent = 'Please select a repository before submitting.';
|
||||||
|
formError.style.display = 'block';
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
formError.style.display = 'none';
|
||||||
|
});
|
||||||
|
|
||||||
|
// Show server-side errors inline on HTMX error responses.
|
||||||
|
document.getElementById('create-issue-form').addEventListener('htmx:responseError', function(evt) {
|
||||||
|
formError.textContent = evt.detail.xhr.responseText || 'An error occurred. Please try again.';
|
||||||
|
formError.style.display = 'block';
|
||||||
|
});
|
||||||
|
})();
|
||||||
</script>
|
</script>
|
||||||
{{end}}
|
{{end}}
|
||||||
|
|||||||
@@ -13,6 +13,10 @@
|
|||||||
<script src="/static/htmx.min.js"></script>
|
<script src="/static/htmx.min.js"></script>
|
||||||
</head>
|
</head>
|
||||||
<body>
|
<body>
|
||||||
|
<header class="top-bar">
|
||||||
|
<span class="top-bar-title">Gitea Mobile</span>
|
||||||
|
<button class="refresh-btn" hx-get="" hx-target="#main-content" hx-swap="innerHTML" aria-label="Refresh">↻</button>
|
||||||
|
</header>
|
||||||
<div class="content" id="main-content">
|
<div class="content" id="main-content">
|
||||||
{{template "content" .}}
|
{{template "content" .}}
|
||||||
</div>
|
</div>
|
||||||
|
|||||||
+44
-1
@@ -47,10 +47,53 @@ body {
|
|||||||
-moz-osx-font-smoothing: grayscale;
|
-moz-osx-font-smoothing: grayscale;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/* Top bar */
|
||||||
|
.top-bar {
|
||||||
|
position: sticky;
|
||||||
|
top: 0;
|
||||||
|
z-index: 100;
|
||||||
|
display: flex;
|
||||||
|
align-items: center;
|
||||||
|
justify-content: space-between;
|
||||||
|
padding: var(--spacing-sm) var(--spacing-lg);
|
||||||
|
padding-top: max(var(--spacing-sm), env(safe-area-inset-top));
|
||||||
|
background: var(--bg-secondary);
|
||||||
|
border-bottom: 1px solid var(--border);
|
||||||
|
}
|
||||||
|
|
||||||
|
.top-bar-title {
|
||||||
|
font-size: var(--font-base);
|
||||||
|
font-weight: 600;
|
||||||
|
color: var(--text-primary);
|
||||||
|
}
|
||||||
|
|
||||||
|
.refresh-btn {
|
||||||
|
background: none;
|
||||||
|
border: none;
|
||||||
|
color: var(--text-secondary);
|
||||||
|
font-size: 1.4rem;
|
||||||
|
padding: var(--spacing-xs) var(--spacing-sm);
|
||||||
|
cursor: pointer;
|
||||||
|
-webkit-tap-highlight-color: transparent;
|
||||||
|
line-height: 1;
|
||||||
|
border-radius: var(--radius-sm);
|
||||||
|
transition: color 0.15s ease, background 0.15s ease;
|
||||||
|
}
|
||||||
|
|
||||||
|
.refresh-btn:active {
|
||||||
|
color: var(--accent-blue);
|
||||||
|
background: var(--bg-tertiary);
|
||||||
|
}
|
||||||
|
|
||||||
|
.refresh-btn.htmx-request {
|
||||||
|
animation: spin 0.6s linear infinite;
|
||||||
|
pointer-events: none;
|
||||||
|
}
|
||||||
|
|
||||||
/* Content area */
|
/* Content area */
|
||||||
.content {
|
.content {
|
||||||
padding: var(--spacing-lg);
|
padding: var(--spacing-lg);
|
||||||
padding-top: max(var(--spacing-lg), env(safe-area-inset-top));
|
padding-top: var(--spacing-lg);
|
||||||
max-width: 640px;
|
max-width: 640px;
|
||||||
margin: 0 auto;
|
margin: 0 auto;
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user