Fixed an incorrect package name in the composer require command which prevented users from installing the module according to the information in the README.md file.

Magento 2.3.3 supports PHP 7.3. Is it possible to add this version in the composer.json?

https://devdocs.magento.com/guides/v2.3/release-notes/release-notes-2-3-3-open-source.html

Preconditions

1. Magento 2.4.4

Steps to reproduce

1. Run composer require-commerce --dev honl/magento2-templatehints

Expected result

1. No error is thrown
2. The module is installed

Actual result

1. The following error is thrown because of PHP ~ 7.1 in the composer.json requirements while Magento 2.4.4 requires PHP >= 8.1:

Package honl/magento2-templatehints has requirements incompatible with your PHP version, PHP extensions and Composer version:  
    - honl/magento2-templatehints 0.2.5 requires php ~7.1 which does not match your installed version 8.1.1.

Run

php magento setup:di:compile

During the step - Interceptors cache generation... return Error:

Ho\Templatehints\Block\Hints\Init
        Incorrect dependency in class Ho\Templatehints\Block\Hints\Init in /Applications/XAMPP/xamppfiles/htdocs/magento2data/vendor/ho-nl/magento2-templatehints/Block/Hints/Init.php
\Magento\Framework\View\Asset\Repository already exists in context object
Total Errors Count: 1
Errors during compilation:
    Ho\Templatehints\Block\Hints\Init
        Incorrect dependency in class Ho\Templatehints\Block\Hints\Init in /Applications/XAMPP/xamppfiles/htdocs/magento2data/vendor/ho-nl/magento2-templatehints/Block/Hints/Init.php
\Magento\Framework\View\Asset\Repository already exists in context object
Total Errors Count: 1
Errors during compilation:
    Ho\Templatehints\Block\Hints\Init
        Incorrect dependency in class Ho\Templatehints\Block\Hints\Init in /Applications/XAMPP/xamppfiles/htdocs/magento2data/vendor/ho-nl/magento2-templatehints/Block/Hints/Init.php
\Magento\Framework\View\Asset\Repository already exists in context object
Total Errors Count: 1

I think in README, we should state which version the module were tested with. But in composer we should allow any PHP version which likely working with instead of fixing specific versions because our module keep out of dated than PHP version update.

For now, please update composer to support any PHP 7.2.x

I have a magento 2.2.6 installation. Shift clicking on a product image opens up the light box view of the product image, not logging any output to the console.

The same seems to be true for buttons having click events.

I get the folowing error after installation via Composer.

Parse error: syntax error, unexpected '?' in /magento/vendor/honl/magento2-templatehints/Plugin/View/LayoutPlugin.php on line 150

I followed the steps in the readme

Slack chat:

@paales: How about using the $proceed closure to generate the view elements content, instead of calling the methods on the elements directly? Just saying, since not calling $proceed can break other around plugins. (edited)

When running in docker, the project is mapped to different folders. It woul be nice to be able to define the root magento path.

Example: In my case, the remote call points to http://localhost:63342/api/file//var/www/html/vendor/magento/module-sales/view/adminhtml/templates/order/view/info.phtml The correct one would be http://localhost:63342/api/file/src/vendor/magento/module-sales/view/adminhtml/templates/order/view/info.phtml

Note there's no extra / and the paths are different in the project. It's impossible to predict, but making it configurable would help.

PS: Congratulations for the amazing work with the module. It helps a lot.

I get

Error: Cannot instantiate abstract class Magento\Framework\Model\ResourceModel\AbstractResource in /var/www/hausfux/m2/vendor/magento/framework/ObjectManager/Factory/Dynamic/Developer.php

You also need to execute php bin/magento setup:di:compile

We have the following HTML

<!--Authentication Pop-Up-->
</div></header><div x-data="initHeaderNavigation()"
 x-init="$watch('activeSubmenuId', value => activateSubmenuMobile(value))"

The module inserts the hint like this

<!--Authentication Pop-Up-->
</div></header><div x-data="initHeaderNavigation()"
 x-init="$watch('activeSubmenuId', value = data-ho-hinttype="block" ....
 @menu-back.window="back"
 class="relative z-50 font-bold bg-gray-200 lg:block lg:z-40"

Reason: We should not parse HTML with regex https://github.com/ho-nl/magento2-Ho_Templatehints/blob/master/Plugin/View/LayoutPlugin.php#L117

Related to Hyvä, because we use Alpine JS there and these notations are common

https://gitlab.hyva.io/hyva-themes/hyva-compat/magento2-ho-templatehints/-/issues/2

Opening in PhpStorm simply doesn't work anymore, regardless the configuration allowing unsigned requests when using docker.

It's just because the FILE is different then the file in my local machine. For example: http://localhost:63342/api/file//var/www/html/vendor/magento/module-catalog/view/frontend/templates/product/view/form.phtml

The correct url would be http://localhost:63342/api/file/src/vendor/magento/module-catalog/view/frontend/templates/product/view/form.phtml

Adding some option to map or replace the url properly would be very nice.

You can't use the extension on the checkout success page. If you add the parameter ?ath=1 to the checkout then it gets automatically removed in checkout success.