cfinke / Feed-Sidebar

The Feed Sidebar is an extension for Firefox that displays the items from your Live Bookmarks in the sidebar.
https://addons.mozilla.org/en-US/firefox/addon/4869/
GNU General Public License v2.0
5 stars 3 forks source link

Feed Sidebar styling to make it look more native #86

Closed Quicksaver closed 10 years ago

Quicksaver commented 10 years ago

Hi Chris,

To be honest, when I sent you the e-mail earlier, I was still using version 8. I saw later that you had already fixed a lot for 8.0.2 (your github repo is still at 8.0.1 by the way), and it looked much better already!

Anyway, here's the changes I mentioned I was working on in my e-mail. Mostly I guided myself by the bookmarks and history sidebars, and tried to make the feeds sidebar resemble them more.

Plus a few "extras":

Obviously, there's a little of my personal taste in there. I hope you like it, but if you want me to, I'll change anything you don't like or don't agree with of course.

I put together a few screenshots, so you can see the changes quickly. They are all ordered as Bookmarks Sidebar - Feed Sidebar (8.0.1) - Feed Sidebar (with this patch).

There's just a couple of things that you should know.

Bug 909820 - New Aero list-item hover styling leaves white line behind - basically, the current places styling (that bookmarks, history, and now feeds use) leaves a little white line when mousing over and out of an item in the sidebar. It's not bothersome (barely noticeable), so they're not giving it a very high priority, but hopefully they'll fix it soon. Bug 983819 - Sidebar style looks bad on Windows 8 - in short, they might change the sidebar style for Windows 8 some day.

I'm following closely both of those bugs, because they also affect my OmniSidebar. When they do, if something needs to be updated in case it's not fixed automatically in the feeds sidebar, I'll be happy to create a new patch for this. I'm just telling you this mostly to give you a heads up.

Let me know what you think or if you need anything for this. Luís Miguel

Screenshots

Windows XP (classic, the colors are a bit dark because it's following the system's colors, and I usually test on a machine with a bit of a dark theme, but I was too lazy to change it). feedscompare-winxp Windows Vista/7 (aero) feedscompare-win7aero Windows 8 feedscompare-win8 Mac OS X feedscompare-mac Linux (Ubuntu) feedscompare-linux

cfinke commented 10 years ago

Thanks, this is awesome. I'll upload it as 8.0.3 today.

Quicksaver commented 10 years ago

Cool. Let me know if you or anyone runs into any theme issues because of these changes. :)