Enhancements #148
Enhancements #148
Conversation
… profile vorhanden
Sprachschalter korrigiert Caching korrigiert CustomPlugin rausgenommen und erklärt wie man sie einbindet Umbau auf ensure und redaxo min. 5.6
Co-Authored-By: dergel <jan.kristinus@yakamara.de>
Co-Authored-By: dergel <jan.kristinus@yakamara.de>
|
@phoebusryan ping.. soll ich mich um die Aktualisierungen und Fehlerkorrekturen allgemein kümmern ? |
|
heute abend schau ich nochmal quer durch den PR. habs am WE nicht geschafft. |
Komme aktuell nicht dazu das Ganze im Detail anzuschauen :-/ |
|
Kein Problem .. ich übernehme dann und baue eine neue Version auf.. Warten noch auf @staabm Feedback und erstelle dann noch eine neue Version |
lang/de_de.lang
Outdated
| @@ -40,8 +40,7 @@ redactor2_profiles_label_linkify_option_true = Ja | |||
| redactor2_profiles_label_linkify_option_false = Nein | |||
| redactor2_profiles_label_imagetag = HTML-Tag um Bilder<br><i>z.B. p oder figure</i> | |||
| redactor2_profiles_label_redactorplugins = Plugins | |||
| redactor2_profiles_label_redactorcustomplugins = Custom Plugins | |||
| redactor2_profiles_label_redactorcustomplugins_help = Kommagetrennt mit der Syntax "name:pfad/zum/plugin.js" | |||
| redactor2_profiles_label_redactorcustomplugins_help = Bitte eigene Plugins über das ProjektAddon einbinden. (-EP PACKAGES_INCLUDED LATE-) | |||
There was a problem hiding this comment.
(-EP PACKAGES_INCLUDED LATE-) was soll dieser hinweis bedeuten?
There was a problem hiding this comment.
das man seine redactor Plugins über diesen EP und mit Status LATE einbinden muss, damit diese verwendet werden können..
There was a problem hiding this comment.
vllt sollte man das in einem satz formulieren. die notation hier wirkt so als könnte man sie nur lesen, wenn man weiß was ein EP ist und wenn man weiß das PACKAGES_INCLUDED ein EP ist und dass EPs auch LATE sein können
(impliziert viel detailliertes wissen, was viele nicht haben)
lang/es_es.lang
Outdated
| @@ -36,8 +36,7 @@ redactor2_profiles_label_linkify = ¿Convertir enlaces automáticamente? | |||
| redactor2_profiles_label_linkify_option_true = Sí | |||
| redactor2_profiles_label_linkify_option_false = No | |||
| redactor2_profiles_label_redactorplugins = Plugins | |||
| redactor2_profiles_label_redactorcustomplugins = Complementos personalizados | |||
| redactor2_profiles_label_redactorcustomplugins_help = Coma separada por la sintaxis "name:path/to/plugin.js" | |||
| redactor2_profiles_label_redactorcustomplugins_help = Include your plugins via your project addon. (-EP PACKAGES_INCLUDED LATE-) | |||
There was a problem hiding this comment.
frag mal gregor, ich glaube dass wir keine englischen keys in anderen sprachen machen. den key sollte man dann vermutlich leer machen, oder rauslöschen (damit ein übersetzer im traduko sieht was fehlt)
There was a problem hiding this comment.
klingt sinnvoll.. ändere ich
| rex_sql::factory()->setQuery('INSERT IGNORE INTO '.rex::getTable('redactor2_profiles').' (`id`, `name`, `description`, `minheight`, `maxheight`, `characterlimit`, `urltype`, `externalurltarget`, `shortcuts`, `linkify`, `imagetag`, `redactor_plugins`) VALUES | ||
| (1, "full", "Standard Redactor-Konfiguration", "300", "800", "0", "relative", "blank", "0", "1", "", "anchorlink,alignment,blockquote,bold,cleaner,clips[Snippetname1=Snippettext1|Snippetname2=Snippettext2],deleted,emaillink,externallink,fontcolor[Weiss=#ffffff|Schwarz=#000000],fontfamily[Arial|Times],fontsize[12px|15pt|120%],format[Absatz Klein=p.small|Absatz Mittel=p.middle|Absatz Gross=p.big],fullscreen,groupheading[1|2|3|4|5|6],grouplink[email|external|internal|media|telephone],grouplist[unorderedlist|orderedlist|indent|outdent],heading1,heading2,heading3,heading4,heading5,heading6,horizontalrule,internallink,italic,media,medialink,orderedlist,paragraph,properties,redo,source,styles[code=Code|kbd=Shortcut|mark=Markiert|samp=Sample|var=Variable],sub,sup,table,telephonelink,textdirection,underline,undo,unorderedlist");'); | ||
|
|
||
| redactor2::createJavascriptFile(); |
There was a problem hiding this comment.
können wir in der install.php schon auf klassen zugreifen, die erst mit dem addon verfügbar werden?
There was a problem hiding this comment.
anscheinend .. habe es getestet
There was a problem hiding this comment.
ggf. war der autoloader von tests von dir schon bestückt. man müsste es testen in einer installation in der das addon noch nie zuvor installiert war (oder den autoloader-cache manuell löschen, wenn das addon nicht installiert ist)
There was a problem hiding this comment.
genauso nochmal getestet .. ging
|
die gewünschten performance benefits sind eingetreten, super. ansonsten hab ich ein paar kommentare hinterlassen, die du nach gusto fixen kannst. |
| return $currentLanguage; | ||
| } | ||
|
|
||
| public static function getLanguages() |
There was a problem hiding this comment.
kommentar, was eine sprache mitbringen muss um hier enthalten zu sein
closes #147