Allon Moritz comments

Results 343 comments of


                                            Allon Moritz

Media Actions - Better UX

Can you not just hide the item text on small screens and then have another look on mobile in a follow up pr?

[4.0] Fix package installer not setting new changelogurl

I have tested this item :white_check_mark: successfully on 054ae224b8aca74997f3a4c6a87e4a6af36082dcThis comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35794.

Support {@inheritdoc}

For example Eclipse does support it and I guess PHPStorm does as well. It makes no sense to copy doc blocks from the base class. If that doc block changes...

[4.x] Filter articles in admin module mod_latest by workflow stage

@MagnusSinger can you rebase the pr to the 4.2-dev branch as it is a new feature?

[4.0] Use getIdentity for plugins

Can you fix the conflicts here? Keep in mind that the service provider plugins have the app already injected. Would love to get this merged.

[4.2] [new feature] TinyMce sample text plugin

See my comments in #36968. Same can be applied here as well.

[4.2] Finder: Enforce Taxonomies to contain a title

Thanks!

Activate all unit tests as default

@Hackwar can you have a look on this pr, as it is ready now?

Activate all unit tests as default

When 4.3 is up to date, then the pr should be fine again. Do you mean this plugin https://github.com/joomla/joomla-cms/pull/38421/commits/31a2677816a22da4d3f6d831a919efed9b93a139?

Fix bug use value 1 as TRUE Workflowcondition Field

But we need a test instruction, so other people can test your pr if it fixes the issue. See #37521 for an example.