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