-
Notifications
You must be signed in to change notification settings - Fork 101
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
Update layout: Color modes, jQuery removal, and more #186
base: master
Are you sure you want to change the base?
Conversation
data/awards.py
Outdated
<a href="https://digital.com/">Digital.com's</a></p> | ||
<p class="award"><a href="%(link)s">Best Database Management Software list</a></p>. | ||
<p>in January 2021.</p> | ||
<a href="https://digital.com/" target="_blank" rel="noopener noreferrer">Digital.com's</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.
You can remove the noreferrer from the awards page
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.
could we have this one as svg ?
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.
Sure. Unfortunately I was not able to find an SVG for Scrutinizer and Weblate 😕
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.
@nijel can you provide a Weblate svg please ?
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.
same, as svg would be great
ping @ nijel to have the asset if needed
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.
This should be the one you have: https://github.com/WeblateOrg/graphics/blob/main/logo-text/Logo-Whitetext.svg
You are welcome to choose a different variant from that repo in case the white text is not the best fit.
pmaweb/templates/_conservancy.html
Outdated
incorporated in New York, donors can often deduct the donation on their USA | ||
taxes. | ||
phpMyAdmin is a member project of the | ||
<a href="https://sfconservancy.org" target="_blank" rel="noopener noreferrer">Software Freedom Conservancy</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.
same, remove noreferrer
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.
Should I just remove noreferrer from all or only where you mentioned?
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.
Honestly, I would say all
id="downloadBtn" | ||
class="btn btn-success" | ||
href="{{ file.get_absolute_url }}" | ||
download |
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 is this attribute for ?
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.
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/a#download
Basically, it tells the browser to treat the URL as a download (and not to navigate there). But it can be removed, apparently this only works for same-origin URLs, and the downloads are on files.phpmyadmin.net
.
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.
Thank you !
I did learn something
{% endif %} | ||
|
||
<div class="d-flex gap-2"> | ||
<a class="btn btn-secondary flex-fill" href="https://demo.phpmyadmin.net/master-config/public/" target="_blank" rel="noopener noreferrer"> |
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.
same, keep the referer
pmaweb/templates/_sponsors.html
Outdated
<div class="p-3 border rounded"> | ||
<div class="row {{ row_cols|default:'row-cols-1 row-cols-md-2' }} g-3"> | ||
<div class="col text-center"> | ||
<a href="https://www.vapehuset.se" target="_blank" rel="noopener noreferrer"> |
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.
this is one of the most critical parts and lines of the website
please remove noreferrer
see bd92a98 and the commit message
<td>250x250 logo on <a href="{% url 'home' %}" target="_blank" rel="noopener noreferrer">home page</a> and <a href="{% url 'sponsors' %}" target="_blank" rel="noopener noreferrer">sponsors page</a></td> | ||
</tr> | ||
<tr> | ||
<td>Platinum</td> | ||
<td>$10,000</td> | ||
<td>250x130 logo on <a href="{% url 'home' %}" target="_blank" rel="noopener noreferrer">home page</a> and <a href="{% url 'sponsors' %}" target="_blank" rel="noopener noreferrer">sponsors page</a></td> | ||
</tr> | ||
<tr> | ||
<td>Gold</td> | ||
<td>$5,000</td> | ||
<td>250x70 logo or text link on <a href="{% url 'home' %}" target="_blank" rel="noopener noreferrer">home page</a> and <a href="{% url 'sponsors' %}" target="_blank" rel="noopener noreferrer">sponsors page</a></td> | ||
</tr> | ||
<tr> | ||
<td>Silver</td> | ||
<td>$2,500</td> | ||
<td>250x70 logo on <a href="{% url 'sponsors' %}" target="_blank" rel="noopener noreferrer">sponsors page</a></td> | ||
</tr> | ||
<tr> | ||
<td>Bronze</td> | ||
<td>$1,000</td> | ||
<td>Text link on <a href="{% url 'sponsors' %}" target="_blank" rel="noopener noreferrer">sponsors page</a></td> |
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 remove all noreferrer on all links of the website to the website itself
pmaweb/templates/base.html
Outdated
</a> | ||
</li> | ||
<li class="nav-item"> | ||
<a class="nav-link" href="https://x.com/phpmya" target="_blank" rel="noopener noreferrer"> |
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.
same, noreferrer does no good
pmaweb/templates/sponsors.html
Outdated
<h3>Bronze Sponsors</h3> | ||
<div class="row mb-3 border rounded p-3"> | ||
<div class="col-12 col-md-6 col-lg-4"> | ||
<a href="https://www.loadview-testing.com" target="_blank" rel="noopener noreferrer">Load View Testing</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.
same
<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24" id="scrutinizer-ci">
<path d="m3.991 19.862.005-.004.002-.002zM22.2 12.943c.007.014.011.029.017.043l.005.008-.022-.051zm-4.304 10.944c-.131.004-.262.005-.393.008l-.003.001.396-.009z"></path>
<path fill="#868E91" d="M22.101 19.055a8.207 8.207 0 0 0 .397-2.55 8.346 8.346 0 0 0-.775-3.519 8.583 8.583 0 0 0-2.307-2.859l-.017-.013a6.833 6.833 0 0 0-2.35-.976l-.045-.008c-.246-.06-4.052-.106-7.812-.121l.001-.002c-3.759-.015-7.472 0-7.532.06-.126.126-.06.798.06 1.35a5.66 5.66 0 0 0 1.834 2.753l.01.008a7.501 7.501 0 0 0 3.264 1.581l.049.009c.366.126 1.044.126 4.598.126 4.237 0 4.916.06 5.221.24.142.107.265.23.369.367l.003.005c.18.24.181.307.181.979s-.066.732-.186 1.044a1.48 1.48 0 0 1-.419.362l-.007.004c-.186.12-.307.119-4.358.179-4.671 0-5.156 0-6.321.432a6.458 6.458 0 0 0-2.463 1.352c-1.163 1.038-1.84 2.202-1.96 3.426l.06.611C3.891 23.962 6.591 24 9.3 24c2.577 0 5.146-.037 7.707-.105l.489-.119c.285-.069.531-.153.766-.256l-.029.011.367-.18a7.768 7.768 0 0 0 3.484-4.242l.017-.054z"></path>
<path fill="#08458E" d="M22.461.12C22.401 0 21.421 0 14.863 0L6.877.06a6.172 6.172 0 0 0-3.732 2.488l-.013.02a6.328 6.328 0 0 0-1.348 2.596l-.009.044-.12.553-.119.857c-.06.799-.06 1.351.12 1.471h5.276c.06 0-.186-.246-.186-.672 0-.738.252-.924.552-1.23.552-.426 2.946-.12 4.729-.246 2.449 0 4.603-.06 4.849-.12 2.701-.427 5.031-2.388 5.524-4.536.121-.547.121-1.105.061-1.165z"></path>
</svg> https://iconscout.com/free-icon/scrutinizer-ci-3521695 Or https://github.com/avmaisak/ngx-simple-icons/blob/develop/ngx-simple-icons/src/lib/icons/scrutinizerci.ts Ref: https://github.com/search?q=%22scrutinizer%22+%22%3Csvg%22&type=code Pick the one you want |
I think one with the name in the logo would be better, to match the others. Also found more things to check:
One more thing, on https://www.phpmyadmin.net/try/ the link to the stable release seems to be broken. It links to https://demo.phpmyadmin.net/STABLE/. |
pmaweb/templates/team.html
Outdated
with databases. One of his passions is to lower the barrier so new contributors can feel comfortable submitting a new pull request and he | ||
enjoys coaching prospective contributors through seeing their work merged. William lives in France. | ||
</p> | ||
<h2>Previous Team Members</h2> |
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.
I've randomly noticed this section while looking and new website screenshots after being mentioned for the Weblate logo, and it seems unfair to me. Listing one randomly chosen previous team member and omitting people who spent years on the project is not the best approach. Better to remove previous members here and keep them only on https://docs.phpmyadmin.net/en/latest/credits.html.
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.
Listing one randomly chosen previous team member and omitting people who spent years on the project is not the best approach.
I think this is mostly due to the fact that Dan never announced leaving the team. So that was me requesting to move him to a new section
But you are right, let's make a nice page for all of them that spent a lifetime maintaining phpMyAdmin
The link should be right after the team members in a visible way
Edit: it would be great to pick some of them an make a page ? or just use credits ?
I think the pull request is ready for review now. I changed the themes page so it doesn't need any extra JavaScript code. I have removed “All” and reversed the list. I noticed that there are still themes for version 2 in An SVG logo for Scrutinizer is still missing, preferably one with white text. Unfortunately I couldn't find anything. Furthermore, should I remove all links to planet.phpmyadmin.net? |
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.
no svg ?
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.
Couldn't find one with white text :(
You should have received a copy of the GNU General Public License | ||
along with this program; if not, write to the Free Software | ||
Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. | ||
You should have received a copy of the GNU General Public License along with this program; if not, write to the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. |
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.
https://www.gnu.org/licenses/old-licenses/gpl-2.0.txt
https://www.fsf.org/blogs/community/fsf-office-closing-party
maybe the text could be updated or linked to the GNU website
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 is a link to https://www.gnu.org/licenses/gpl-2.0.html already, should I still update the text or just remove it?
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 diff looks good
Thank you for all the work !
Yes, please do in a separate commit. I can cherry-pick it right away
Very cool !
Removed from where ? the files. server ?
Okay, can you give me one that is okay and I can change the text color with Inkscape ? |
Alright, will do! Should I also remove all the code and the migration?
In the version list on the Themes page there is no tab for version 2. Since “All” is now gone, these are no longer visible. Or should I add version 2 there?
Unfortunately I could not find any SVG from Scrutinizer where text is included. |
This improves the phpMyAdmin website. It updates the layout, many of the pages and cleans everything up a bit.
Changes
color-modes.js
script comes from the Bootstrap docs.colorbox
withbs5-lightbox
.Screenshots
Home page
Home page (light)
Download modal
Docs page
Team page
404 page