Ticket #1172 (closed enhancement: fixed)

Opened 9 months ago

Last modified 7 months ago

[LangMarks] make available languages configurable

Reported by: guest Owned by:
Priority: normal Milestone: 0.95
Component: Plugins Version: trunk
Severity: normal Keywords: LangMarks
Cc: rimas.kudelis@…

Description

Currently, to change the list of languages available in LangMarks plugin, one needs to edit lang-marks.js.

Instead, it should be configurable via Xinha.Config, just like many other plugins are.

Attachments

LangMarks.diff (6.9 kB) - added by guest 8 months ago.
New patch. Takes care of CSS, and sets same class "haslang" for all spans it creates.
lang-marks-lang.diff (410 bytes) - added by guest 8 months ago.
Incremental patch

Change History

Changed 9 months ago by guest

  • owner deleted
  • component changed from Xinha Core to Plugins

Changed 8 months ago by guest

New patch. Takes care of CSS, and sets same class "haslang" for all spans it creates.

Changed 8 months ago by guest

With this new patch, the img/ subdirectory can be entirely removed from the plugin directory.

Changed 8 months ago by guest

  • summary changed from LangMarks plugin should be configurable through Xinha.Config to [patch] LangMarks plugin should be configurable through Xinha.Config

Changed 8 months ago by ray

  • type changed from defect to enhancement
  • milestone set to 0.96

Changed 8 months ago by guest

What is the process to get this checked-in?

Changed 8 months ago by ray

  • status changed from new to closed
  • summary changed from [patch] LangMarks plugin should be configurable through Xinha.Config to [LangMarks] make available languages configurable
  • resolution set to fixed
  • milestone changed from 0.96 to 0.95

rev [911]: cool idea generating the language sign with CSS. Of course this doesn't work with IE, but I think the plugin is way more usable now, so thats worth it.

Changed 8 months ago by guest

Thank you. :) I stole the idea and from Mozilla's webpages. :)

Just for the record, it's rev [991] actually.

Changed 8 months ago by guest

  • status changed from closed to reopened
  • resolution deleted

Changed 8 months ago by guest

Incremental patch

Changed 8 months ago by guest

Please apply lang-marks-lang.diff on top. You didn't have to capitalize [lang] in CSS, it should be lowercase (because the attribute is lowercase).

By the way, (the idea just hit me) would it be possible to use 'xml:lang' instead of 'lang', which is deprecated in xhtml? That would be a subject for a new ticket, I guess..

Changed 8 months ago by ray

  • status changed from reopened to closed
  • resolution set to fixed

rev [995]: thanks

Currently your idea with xml:lang makes not much sense, as Xinha is has to work for both HTML and XHTML and I think it's better to have an attribute that's deprecated in the one dialect than one that doesn't even exist in the other. In the long run we should be able to switch doctypes globally, but that's surely stuff for another ticket...

Changed 8 months ago by guest

I filed #1189 regarding xml:lang. Ray, if you're reading this, would you please take a look at the suggestion I made there?

Changed 7 months ago by guest

Why isn't this available in the latest nightly?

Note: See TracTickets for help on using tickets.