-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
initial work on gui #2
Conversation
e0ccc29
to
dcf5ed5
Compare
634cbe8
to
d9002a0
Compare
.gitignore
Outdated
@@ -1,24 +1,8 @@ | |||
**/*.log |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add those back in
views/auth/login_templ.go
Outdated
}) | ||
} | ||
|
||
func LoginBaseLayout() templ.Component { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Refactor this function you can split it into the writer and variables, also use string interpolation to fill in the variables
views/auth/login_templ.go
Outdated
}) | ||
} | ||
|
||
func LoginForm() templ.Component { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Refactor
views/auth/login_templ.go
Outdated
} | ||
ctx = templ.ClearChildren(ctx) | ||
for _, err := range errors { | ||
templ_7745c5c3_Err = components.WarningNofication(err.Error()).Render(ctx, templ_7745c5c3_Buffer) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
err.Error() could expose sensitive information
views/components/headbar_templ.go
Outdated
templ_7745c5c3_Var1 = templ.NopComponent | ||
} | ||
ctx = templ.ClearChildren(ctx) | ||
_, templ_7745c5c3_Err = templ_7745c5c3_Buffer.WriteString("<header class=\"w-full items-center bg-slate-50 py-2 px-6 hidden sm:flex\"><div class=\"w-1/2\"></div><div x-data=\"{ isOpen: false }\" class=\"relative w-1/2 flex justify-end\"><button @click=\"isOpen = !isOpen\" class=\"realtive z-10 w-12 h-12 rounded-full overflow-hidden border-4 border-gray-400 hover:border-blue-600 focus:border-blue-600 focus:outline-none\"><img src=\"https://source.unsplash.com/uJ8LNVCBjFQ/400x400\"></button> <button x-show=\"isOpen\" @click=\"isOpen = false\" class=\"h-full w-full fixed inset-0 cursor-default\"></button><div x-show=\"isOpen\" class=\"absolute w-32 bg-white rounded-lg shadow-lg py-2 mt-16 z-50\"><a href=\"#\" class=\"block px-4 py-2 account-link hover:text-blue-600\">Account</a> <a href=\"#\" class=\"block px-4 py-2 account-link hover:text-blue-600\">Settings</a> <a href=\"#\" class=\"block px-4 py-2 account-link hover:text-blue-600\">Sign Out</a></div></div></header>") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
refactor
views/components/navbar_templ.go
Outdated
templ_7745c5c3_Var4 = templ.NopComponent | ||
} | ||
ctx = templ.ClearChildren(ctx) | ||
_, templ_7745c5c3_Err = templ_7745c5c3_Buffer.WriteString("<aside class=\"relative bg-blue-600 dark:bg-blue-950 h-screen w-64 hidden sm:block shadow-xl\"><div class=\"p-6\"><a href=\"index.html\" class=\"text-white text-3xl font-semibold uppercase hover:text-gray-300\"><img src=\"https://github.com/COSSAS/SOARCA/raw/development/img/soarca-logo.svg\" class=\"w-30 py-3 md:py-0 g-image\"></a> <button class=\"w-full bg-white text-blue-800 font-semibold py-2 mt-5 rounded-br-lg rounded-bl-lg rounded-tr-lg shadow-lg hover:shadow-xl hover:bg-gray-300 flex items-center justify-center\"><i class=\"fas fa-plus mr-3\"></i> New Report</button></div><nav class=\"text-white text-base font-semibold pt-3\">") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
refactor
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With pain in my heart I will this due to having no tests
No description provided.