NPM Resource Changed ? ? Pending

User tests: Successful: Unsuccessful:

avatar rdeutz
rdeutz
24 Mar 2021

Pull Request for Issue #19429 last bits and #30018 but changed my mind about the buttons.

Summary of Changes

  • Tries to make install from web look like the other Joomla4 backend stuff
  • Some javascript magic for the clean button
  • Added a legend like the other tabs

Testing Instructions

  • apply patch
  • npm build:css
  • npm build:js
  • look if search works as expected

Actual result BEFORE applying this Pull Request

grafik

Expected result AFTER applying this Pull Request

grafik

grafik

grafik

Note: Seems the language pack doesn't contain a translation for the install from web tab.

Documentation Changes Required

don't think so

avatar rdeutz rdeutz - open - 24 Mar 2021
avatar rdeutz rdeutz - change - 24 Mar 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 24 Mar 2021
Category JavaScript Repository NPM Change Front End Plugins
avatar rdeutz rdeutz - change - 24 Mar 2021
Title
[4.0] Make Webinstall looking better
[4.0] Make Install from Web looking better
avatar rdeutz rdeutz - edited - 24 Mar 2021
avatar rdeutz rdeutz - change - 24 Mar 2021
The description was changed
avatar rdeutz rdeutz - edited - 24 Mar 2021
avatar jwaisner jwaisner - test_item - 25 Mar 2021 - Tested successfully
avatar jwaisner
jwaisner - comment - 25 Mar 2021

I have tested this item successfully on a82dbe8

Thank you for the work on cleaning this up. Works as expected and looks much better.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/32851.

avatar infograf768
infograf768 - comment - 25 Mar 2021

Note: Seems the language pack doesn't contain a translation for the install from web tab.

This string is new in j4, therefore not avalable in j3 packs
PLG_INSTALLER_WEBINSTALLER_TAB_LABEL="Install from Web"

In J3 it was
COM_INSTALLER_INSTALL_FROM_WEB="Install from Web"

avatar brianteeman
brianteeman - comment - 25 Mar 2021

Everything looks great.

Is it expected that it is not possible to select an extension to install when using the keyboard?

avatar rdeutz
rdeutz - comment - 25 Mar 2021

Is it expected that it is not possible to select an extension to install when using the keyboard?

short answer: yes

To add this we need to change the install-from-web server, because we only show the HTML. This is more or less a mistake in the design of the functionality. I looked into this but this is a huge task.

My changes here are more cosmetic and make sure it looks like the other parts of Joomla4.

avatar brianteeman brianteeman - test_item - 25 Mar 2021 - Tested successfully
avatar brianteeman
brianteeman - comment - 25 Mar 2021

I have tested this item successfully on a82dbe8


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/32851.

avatar drmenzelit drmenzelit - close - 25 Mar 2021
avatar drmenzelit drmenzelit - merge - 25 Mar 2021
avatar drmenzelit drmenzelit - change - 25 Mar 2021
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-03-25 10:08:10
Closed_By drmenzelit
Labels Added: NPM Resource Changed ? ?
avatar drmenzelit
drmenzelit - comment - 25 Mar 2021

Thanks

Add a Comment

Login with GitHub to post a comment