Seekra/frontend
3
2
Fork 0
generated from Seekra/repository-template
Code Issues 4 Pull Requests Actions Packages Projects 1 Releases Wiki Activity

Add NotFound.vue #48 #50

Merged
jakob.scheid merged 22 commits from feature/not-found-page into main 2026-05-14 16:21:30 +02:00
Conversation 8 Commits 22 Files Changed 3
+11 -1
1 changed files with 11 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit cf7525b32a - Show all commits
src/views/NotFound.vue
+11 -1
View File
@@ -1,11 +1,21 @@
<template> <template>
<span class="error-message"> <div class="not-found">
jakob.scheid marked this conversation as resolved Outdated
jakob.scheid commented 2026-05-14 13:20:28 +02:00
Outdated
Review
Copy Link
Copy Source

Do not use a container .search-content in a not found page.

Do not use a container *.search-content* in a not found page.
<span class="error-message">
jakob.scheid marked this conversation as resolved Outdated
jakob.scheid commented 2026-05-14 13:22:08 +02:00
Outdated
Review
Copy Link
Copy Source

The not found page should be minimal, so omit the header. The navigation bar already contains Seekra.

The not found page should be minimal, so omit the header. The navigation bar already contains *Seekra*.
The page you are looking for does not exist. Please check the URL or return to the search page. The page you are looking for does not exist. Please check the URL or return to the search page.
</span> </span>
jakob.scheid marked this conversation as resolved Outdated
jakob.scheid commented 2026-05-14 13:21:22 +02:00
Outdated
Review
Copy Link
Copy Source

No HTTP status code. That is confusing for people who are not familiar with technology.

No HTTP status code. That is confusing for people who are not familiar with technology.
<RouterLink to="/" id="link">Back to Search</RouterLink> <RouterLink to="/" id="link">Back to Search</RouterLink>
jakob.scheid marked this conversation as resolved Outdated
jakob.scheid commented 2026-05-14 13:19:17 +02:00
Outdated
Review
Copy Link
Copy Source

Do not write ERROR uppercase.

Do not write *ERROR* uppercase.
</div>
</template> </template>
<style scoped> <style scoped>
.not-found {
text-align: center;
padding-top: 80px;
display: flex;
jakob.scheid marked this conversation as resolved Outdated
jakob.scheid commented 2026-05-14 13:18:20 +02:00
Outdated
Review
Copy Link
Copy Source

Copying styles is a very bad idea. Nothing should be redundant.

Copying styles is a very bad idea. Nothing should be redundant.
flex-direction: column;
align-items: center;
gap: 12px;
}
.error-message{ .error-message{
margin: 0; margin: 0;
font-weight: 600; font-weight: 600;