-
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
Paranoid GUI #378
Paranoid GUI #378
Conversation
How much third-party code is on your branch's history? Not just in the latest commit. |
{ | ||
"name" : "paranoid-gui", | ||
"description" : "Paranoid GUI", | ||
"version" : "0.0.1", |
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.
Nitpick, but this should be 0.1.0
for the first prerelease version.
What's the status of this PR? |
Consider using CSS variables for things that share the same colour |
All error messages appear to be alerts, which gets quite annoying after a while. |
This branch should be rebased off master in order to fix those failing builds. EDIT: (done) |
@GoldenBadger |
01dd4fd
to
308794f
Compare
what is the story with the pictures in the commit :) |
<body> | ||
<div id="header"> | ||
<div class="logo"> | ||
<a onclick="rowClicked(-1)">Paranoid<span>Filesystems</span></a> |
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.
what does -1 mean in this instance?
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.
-1 is the index of the home section
Visually the code looks file. I'm a bit rusty on my js so I'm not really sure about how its implemented. |
Init, Mount, Unmount and Delete functionality implemented in GUI