-
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
Styling #4
Conversation
Database access
- changed data table back from CSS grid to table - adjusted tags and structure - the name of the person now is a heading (caption) not a table row - added styling - added Javascript to only show a preview of the content if it is too large
- linked people.css to people.php (in head) - added comment regarding `themecolor`
fixed commit
- themecolor now is determined automatically, though it can be manually overridden - spacing was automatically adjusted
__DIR__ returns the directory of the file
reformatting, relinking, added new rows
Non-Binary now supported
some minor fixes
added fonts and Licenses
config.php
Outdated
'password' => "password", | ||
'DBname' => "worldDB", | ||
); | ||
?> |
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.
?> |
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.
why is this needed?
scripts/connection.php
Outdated
die ( "Connection failed: " . $conn->connect_error ); | ||
} | ||
return $conn; | ||
?> |
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.
?> |
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.
why is this needed?
<?php | ||
$config = require ($_SERVER['DOCUMENT_ROOT'] . "../config.php"); | ||
// Create connection | ||
$conn = new mysqli ( $config ['servername'], $config ['username'], $config ['password'], $config ['DBname'] ); |
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.
consider wrapping the whole thing in a class and making the connection static
Co-authored-by: Ein Typ <34921967+eintyp@users.noreply.github.com>
people.php
Outdated
<?php | ||
// Main color of color sheme (appearance image) and version with alpha = 0 | ||
if (file_exists($imgpath)) { | ||
include_once (__DIR__ . "/scripts/themecolor.php"); |
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.
include_once (__DIR__ . "/scripts/themecolor.php"); | |
require_once (__DIR__ . "/scripts/themecolor.php"); |
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.
the script is not critical for operation
Co-Authored-By: Ein Typ <34921967+eintyp@users.noreply.github.com>
better than if else chain Co-authored-by: Ein Typ <34921967+eintyp@users.noreply.github.com>
Co-Authored-By: Ein Typ <34921967+eintyp@users.noreply.github.com>
to match php convention
Co-Authored-By: Ein Typ <34921967+eintyp@users.noreply.github.com>
Co-authored-by: Ein Typ <34921967+eintyp@users.noreply.github.com>
Co-authored-by: Ein Typ <34921967+eintyp@users.noreply.github.com>
Some style changes