Project

General

Profile

Bug #6323

Improve the CSS of the boxed titles

Added by sajolida about 4 years ago. Updated over 3 years ago.

Status:
Resolved
Priority:
Low
Assignee:
-
Category:
-
Target version:
-
Start date:
09/29/2013
Due date:
% Done:

30%

QA Check:
Feature Branch:
Type of work:
Website
Blueprint:
Easy:
Yes
Affected tool:

Description

The boxed titles on the website don't fit well in the current color scheme of the website and don't stand out enough in the structure of the page. Those can be seen on:
  • the homepage: "news", "security"
  • the support page: "Search the documentation", "Troubleshooting", "Get in touch with us"
  • the troubleshooting page: "Report an error", "Request a feature", "Talk"
  • the contribute page: "Design", "Tools", "Other development pages", "Talk with us"

This styling can probably be simply removed from the CSS, and the style of the columns on the homepage, and troubleshooting redefined. While doing this change some HTML code or HTML pages can be converted to markdown. Markdown is lighter to work with and more resilient to dead links. Some titles also need to be converted to use sentence-style capitalization as the rest of the website.

2.jpg View - screenshot (175 KB) esperal, 04/13/2014 12:08 PM

Associated revisions

Revision 1dbbd354 (diff)
Added by Tails over 3 years ago

refs #6323 integrating esperals changes on boxed titles to css

Revision a3d62132 (diff)
Added by Tails over 3 years ago

refs #6323 integrating esperals changes on boxed titles to css

Revision 581f21fe (diff)
Added by Tails over 3 years ago

refs #6323 smaller title padding

Revision 121c8f27 (diff)
Added by Tails over 3 years ago

refs #6323 small improvement

Revision efe216a2 (diff)
Added by Tails over 3 years ago

refs #6323 integrating esperals changes on boxed titles to css

Revision 11cfb7cf (diff)
Added by Tails over 3 years ago

refs #6323 integrating esperals changes on boxed titles to css

Revision 1607379c (diff)
Added by Tails over 3 years ago

refs #6323 smaller title padding

Revision f5d0dea6 (diff)
Added by Tails over 3 years ago

refs #6323 small improvement

History

#1 Updated by sajolida about 4 years ago

  • Priority changed from Normal to Low

#2 Updated by esperal over 3 years ago

Yeah I agree with those boxes. I just changed few bits in the local.css to give it more importance on the pages. See the screenshot file attached and if you're happy do the blow...

In the local.css file on line: 396

1. Change the color:#96C35A; to #fff
and add background: #00AA00;

I also removed the text-shadow because it looked strange, soft.

#3 Updated by intrigeri over 3 years ago

  • Assignee set to u

u., do you want to comment on the proposed changes? Please reassign to nobody once you're done, or if you don't feel like reviewing it.

#4 Updated by intrigeri over 3 years ago

  • QA Check set to Ready for QA

#5 Updated by u over 3 years ago

Thanks intrigeri.
Yeah, i prefer the new version of the boxed titles :)

#6 Updated by u over 3 years ago

Shall i apply these modifications myself and ask for a pull?

#7 Updated by intrigeri over 3 years ago

Shall i apply these modifications myself and ask for a pull?

Yes, please.

#8 Updated by u over 3 years ago

  • % Done changed from 0 to 50

#9 Updated by u over 3 years ago

  • % Done changed from 50 to 30

created a separate ticket for converting pages to markdown here: https://labs.riseup.net/code/issues/7086

#10 Updated by u over 3 years ago

Sajolida, i could not find any titles on which to apply this:
"Some titles also need to be converted to use sentence-style capitalization as the rest of the website." Do you have an example?

#11 Updated by sajolida over 3 years ago

Yes, the "news" and "security" titles on the homepage are all lowercase.

Bonus point if after doing that you somehow rescue the translations from all the PO files, and fix the capitalization there. I think in that case it should be doable even without knowing the precise capitalization rule of each language.

#12 Updated by sajolida over 3 years ago

  • Status changed from Confirmed to Resolved
  • Assignee deleted (u)
  • QA Check deleted (Ready for QA)

Also available in: Atom PDF