- The "Subscribe to: This post" link is split across two lines and has an orphan.
- The orphan "post" is indented more than the item above it.
- The link icons should not be cut off.

I was able to resolve the overflow/ links being cut off issue, but not the others with this change.

Only local images are allowed.

Comments

ezra-g’s picture

Only local images are allowed.

sheena_d’s picture

Status: Needs work » Needs review

Ezra,

I don't think your patch will actually solve the problem of the item text wrapping, it just happens to solve the symptom in this one particular case (i.e. items with different lengths of characters are going to wrap at different places, so adjusting the padding will only solve the issue in some cases where the breaks between items happent to fall in the right place).

Anyhoo, I don't have a set up of Commons handy to be able to roll a patch, but here is basically the fix for all three issues:

/*commons_origins-style.css at line 87*/

.item-list ul li,
.node .inner ul li,
.panel-pane .inner ul li,
.block .content ul li {
-  background: transparent url(../images/gray-arrow-right.png) no-repeat 0 4px;
+  background: transparent url(../images/gray-arrow-right.png) no-repeat left center;
  list-style-image: none;
  list-style: none;
  margin-bottom: 8px;
  padding-left: 22px;
  text-indent: 0;
}


/*commons_origins-style.css at line 896*/

.node .inner ul.menu li a,
.block .content ul.menu li a,
-.panel-pane .inner ul.menu li aa {
+.panel-pane .inner ul.menu li a {
-  background: transparent url(../images/gray-arrow-right.png) no-repeat 0 1px;
+  background: transparent url(../images/gray-arrow-right.png) no-repeat left center;
	list-style-image: none;
	list-style: none;
	margin: 0 0 8px 0;
	padding: 0 0 0 22px;
	text-indent: 0;
+  whitespace: no-wrap;
}

+.node .inner ul.menu,
+.block .content ul.menu,
+.panel-pane .inner ul.menu {
+  margin-left: 0;
+}

Also, there's an extra typo fix in there :)

Just tested in Firebug, so may need more review.

KrisBulman’s picture

Just installed commons, I will roll this as a patch momentarily.

ezra-g’s picture

StatusFileSize
new1.28 KB

Awesome - thanks, Sheena!

This seems to resolve the overflow issue but not the wrapping issue or the inconsistent indentation issue. I've rolled this as a patch to make sure I've got it right.

Thanks again for jumping in here - great to see you in the Commons queue :).

KrisBulman’s picture

StatusFileSize
new1.28 KB

Here is a patch for the above code. whitespace should actually be white-space (fixed in patch_

KrisBulman’s picture

too fast for me!

i will check out the other issues

ezra-g’s picture

Thanks for jumping in, KrisBulman! Actually, my patch missed your pont about white-space, so thanks for re-rolling.

However, it seems like this still doesn't resolve the issue for me. Can you confirm?

KrisBulman’s picture

just checking now, I just re-rolled the patch and fixed the whitespace, but didn't actually check the site.

Where can i access these node link menus in the origins theme? I haven't used commons before. :p

ezra-g’s picture

You can see it on the default discussion node that ships with Commons at discussion/jumpstarting-our-community .

I'm in IRC in #drupal-commons if you want to coordinate at all :).

KrisBulman’s picture

StatusFileSize
new1.38 KB

re-rolled patch with fixes for all issues. give it a try now.

PS noticed a white space bug where your icons are occasionally mashing up against separator pipes. looking into that one.

KrisBulman’s picture

StatusFileSize
new1.38 KB

k, so the icons mashing up against the pipes is definitely the "Extra Whitespace in List Links bug". it's one that's not usually noticed with inline links unless you are doing tight pixel sensitive designs.. and especially with icon links. There are some negative text-indent hacks that can fix this, etc.. but i prefer floated left list items in most cases.

Changing the menus over to floated left menus will actually cause another problem though, the pipes will become the same size as the line-height, which doesn't improve the design very much in this case.

As a quick fix, I just left changed the location of the icon in the list to be moved over to the right by 3px.

last patch re-roll unless you notice any other issues.

ezra-g’s picture

Status: Needs review » Reviewed & tested by the community

Here's what this looks like to me:

Only local images are allowed.

[edit] Ultimately, we should consider moving the subscribe links into a widget, perhaps like Notifications extra as suggested by IceCreamYou, but this solves the present spacing issue. I think this is RTBC.

Huge thanks, KrisBulman and sheena_d!

cpliakas’s picture

Excellent work, guys! Very much in need of this. Confirming that the patch in #11 works great.

ezra-g’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.