Capacity CSS style: follow new style of api.kde.org
AbandonedPublic

Authored by ochurlaud on Feb 23 2016, 5:19 PM.

Details

Summary

I changed the style and tested *only* on l10n. I don't know what other website currently use that so I don't know if it can break something else.

For l10n, some css code must be also changed in its own repo. I'm creating the diff right away.

Diff Detail

Lint
Lint Skipped
Unit
Unit Tests Skipped
ochurlaud updated this revision to Diff 2442.Feb 23 2016, 5:19 PM
ochurlaud retitled this revision from to Capacity CSS style: follow new style of api.kde.org.
ochurlaud updated this object.
ochurlaud edited the test plan for this revision. (Show Details)
ochurlaud added reviewers: ltoscano, aacid.
ochurlaud updated this revision to Diff 2446.Feb 23 2016, 5:51 PM

Use free fonts

What repository is this for? You didn't specify it in the review.

aacid edited edge metadata.Feb 23 2016, 11:06 PM

Which repository is this patch foor?

@aacid this is for the git repo of the capacity framework

The capacity repository is not in Phabricator (and neither is any other website repo, apparently). KDE has not yet "switched to Phabricator", only a few projects and repositories are available.

I assume patches should still be sent via Reviewboard for now,

I don't think it's a good idea changing these values here directly in css.php since will affect all other sites using the "newlayout" layout of capacity.

As far as i can see it there's we have two options:

  • We can create a "newnewlayout" in capacity if we're planing to move more sites from "newlayout" to "newnewlayout"
  • We can just make this changes in css.inc of l10n web if this changes are only for this particular "newlayout" user
ochurlaud updated this revision to Diff 2462.Feb 25 2016, 7:15 AM
ochurlaud edited edge metadata.

I copied the newmayout to newlayout2 and created a new .css file instead. In l10n, the only other change will be in sites.inc

Better now?

I'd prefer if the new file was called css_newlayout2.php instead of css_new.php

includes/newlayout2/template-header.inc
28

This should still be css.inc no?

ochurlaud updated this revision to Diff 2468.Feb 25 2016, 10:46 PM

@aacid: updated what you said.

ochurlaud updated this revision to Diff 2469.Feb 25 2016, 10:51 PM
ochurlaud marked an inline comment as done.

Errors in the previous diff. Sorry

I have the feeling the left bar grows a bit too much http://i.imgur.com/dZWHUeQ.png

Can that be limited? Or is that good and it's just me that have not much idea about "what looks nice" :D

@aacid What did you think about http://api.kde.org/frameworks-api/frameworks5-apidocs/ ?

I'm just putting the same style. I can reduce and so on, but I would need more precise directions... If you think the current online version is better, it can stay like it is.

@aacid What did you think about http://api.kde.org/frameworks-api/frameworks5-apidocs/ ?
I'm just putting the same style. I can reduce and so on, but I would need more precise directions... If you think the current online version is better, it can stay like it is.

The thing is that page was already very wide before (or so says http://web.archive.org/web/20150905161257/http://api.kde.org/frameworks-api/frameworks5-apidocs/ )

Anyway as said i'm not the one that has the better visual taste so i may be just wrong.

Did anyone suggest you to do this change or you did it on your own? If someone suggested it maybe you can point them to my screenshot and the current code so they can see if they like it more?

In D1011#19514, @aacid wrote:

@aacid What did you think about http://api.kde.org/frameworks-api/frameworks5-apidocs/ ?
I'm just putting the same style. I can reduce and so on, but I would need more precise directions... If you think the current online version is better, it can stay like it is.

The thing is that page was already very wide before (or so says http://web.archive.org/web/20150905161257/http://api.kde.org/frameworks-api/frameworks5-apidocs/ )

Anyway as said i'm not the one that has the better visual taste so i may be just wrong.

Did anyone suggest you to do this change or you did it on your own? If someone suggested it maybe you can point them to my screenshot and the current code so they can see if they like it more?

Sorry for the harsh answer.. I picked the task myself because it seems to me that it doesn't look good. I changed the code, and I'm not really sure what direction I should go. Thanks for reviewing: I'll try to think about the size (my screen is only 14''. so on a bigger one it might not render as well as for me). I'll come back here when I have new ideas.

It was a hard day. Time to sleep for me.

In D1011#19514, @aacid wrote:

@aacid What did you think about http://api.kde.org/frameworks-api/frameworks5-apidocs/ ?
I'm just putting the same style. I can reduce and so on, but I would need more precise directions... If you think the current online version is better, it can stay like it is.

The thing is that page was already very wide before (or so says http://web.archive.org/web/20150905161257/http://api.kde.org/frameworks-api/frameworks5-apidocs/ )

Anyway as said i'm not the one that has the better visual taste so i may be just wrong.

Did anyone suggest you to do this change or you did it on your own? If someone suggested it maybe you can point them to my screenshot and the current code so they can see if they like it more?

Sorry for the harsh answer.. I picked the task myself

When you say you picked the task you mean you said "this looks ugly, let's make it look nicer" and not "i went into a list someone made of web pages and picked one from there", right?

because it seems to me that it doesn't look good. I changed the code, and I'm not really sure what direction I should go. Thanks for reviewing: I'll try to think about the size (my screen is only 14''. so on a bigger one it might not render as well as for me). I'll come back here when I have new ideas.

As said i have not the most of the visual/design skills, my input is not the best, you may want to seek some opinions in https://forum.kde.org/viewforum.php?f=285

In D1011#19663, @aacid wrote:

When you say you picked the task you mean you said "this looks ugly, let's make it look nicer" and not "i went into a list someone made of web pages and picked one from there", right?

I was using a lot api.kde.org, found it ugly, so worked on this. Then I took in order the websites having the same style to progressively unify all theses websites.

I'll check the forum post. Thank you for all your advice.

New topic here:
https://forum.kde.org/viewtopic.php?f=285&t=131263

I already sent a mail on the mailinglist last week: no answer yet.

ochurlaud updated this revision to Diff 2724.Mar 11 2016, 9:39 PM

Updated width (1200px max)

ochurlaud updated this revision to Diff 2725.Mar 11 2016, 10:44 PM

Better icons + change font

imalchow edited edge metadata.Mar 11 2016, 11:37 PM

I have added some inline comments. not sure how that works....

css_newlayout2.php
222

make this
margin: 4px 0px 6px 18px;

includes/newlayout2/template-top1.inc
12

make the width 35px, along with the css change it makes the logo a bit more prominent.

ochurlaud updated this revision to Diff 2730.Mar 12 2016, 8:50 AM
ochurlaud edited edge metadata.

Update with neverendigo's ideas

the latest addition didn't seem to have made it into the diff.
But with that i guess this qualifies as a valid addition to the repo. There is no functional change on capacity anyway, just a new theme, which needs to be switched to in the site's config.

aacid added a comment.Mar 13 2016, 9:17 PM

the latest addition didn't seem to have made it into the diff.
But with that i guess this qualifies as a valid addition to the repo. There is no functional change on capacity anyway, just a new theme, which needs to be switched to in the site's config.

Sure, but the idea here is that someone has to agree this looks better than the other layout, otherwise what's the point of adding it.

I just can't be the judge if it looks better or not since i'm not very expert on that side.

ochurlaud marked an inline comment as done.Mar 14 2016, 11:09 AM
In D1011#21615, @aacid wrote:

the latest addition didn't seem to have made it into the diff.
But with that i guess this qualifies as a valid addition to the repo. There is no functional change on capacity anyway, just a new theme, which needs to be switched to in the site's config.

Sure, but the idea here is that someone has to agree this looks better than the other layout, otherwise what's the point of adding it.

I just can't be the judge if it looks better or not since i'm not very expert on that side.

Ken (who deals with the kde.org website) and neverendigo already told me it was better with my modifications. I push it on the website, and then wait for @ltoscano to accept the update for the l10n website.

ochurlaud abandoned this revision.Mar 14 2016, 11:57 PM