Fork me on GitHub
Subscribe 4

Ticket #437 (fixed enhancement)

Add link separator to separate post links

  • Created: 2011-05-15 18:08:39
  • Reported by: Koos
  • Assigned to: Reines
  • Milestone: 1.4.6
  • Component: styling
  • Priority: low

Style developers should have the option of using the link separators to separate links in posts. They were removed in fluxbb 1.4 for some reason. I propose that a span element is then added to the link separators, and one can just make it invisible if they are not needed, e.g. in the Air style. I've attached a patch that show my recommendation. The patch for the Oxygen style I uploaded earlier includes a necessary fix so that the link separators will display correctly when using the Oxygen style.

E.g. for the Technetium I think it would look much better if link separators are used:

ix5mb8.jpg

35jbar4.png

History

Koos 2011-05-15 18:09:52

  • Uploaded patch sep.patch. (view)

Reines 2011-05-16 08:05:34

  • Component set to styling.
  • Milestone set to 1.4.6.
  • Priority changed from normal to low.
  • Type changed from bug to enhancement.

This should be possible using pure CSS, without the need to edit any markup... I think?

quy 2011-05-19 00:10:09

It is definitely possible with CSS only. Personally, I like without separators (less clutter). With Technetium, add padding between options to make it more readable.

Koos 2011-05-24 21:57:15

Yes, it can be done in css. The oxygen like styles look better with link separators.

css link separators can be done like this:

Find:
.pun .postfootright li {
    display: inline;
    }

After, add:

.pun .postfootright li:before {
    content: " | ";
    }

.pun .postfootright li:first-child:before {
    content: "";
    }

taylorchu 2011-05-25 02:21:58

I dont think other developers are going to support old punbb style since 2.0 styles are coming in.

http://fluxbb.org/development/core/tickets/447/

Reines 2011-06-05 19:11:59

  • Owner set to Reines.
  • Status changed from open to fixed.