Pedro Martins

Hello @alihamidali Thanks for the contribution, and sorry we can't merge it. We are working on a solution in #4623 so I will be closing this one to avoid confusions....

@tabacitu I need your opinion on this before I commit any more time here. There is also this comment: https://github.com/Laravel-Backpack/CRUD/issues/4270#issuecomment-1192525886

@tabacitu I don't think it has any influence in `show` because we include the view directly without using that function. That function is only used for datatables columns in `getEntriesAsJsonForDatatables()`...

@tabacitu I have only one concern that I found and mentioned earlier, I think it's no biggie, but I need approval to have peace of mind: >The only downside I...

@ziming thanks for the contribution. How about `temporarySignedUrl()` ? Since we are doing changes here, shouldn't we tackle those too ? Cheers

> Looking at these 2 files, I did not see temporarySignedUrl() > > Only temporaryUrl() > > https://github.com/laravel/framework/blob/9.x/src/Illuminate/Filesystem/FilesystemAdapter.php > > https://github.com/laravel/framework/blob/9.x/src/Illuminate/Filesystem/AwsS3V3Adapter.php > > I think temporarySignedUrl() is for normal laravel...

Hello @joelmellon sorry for the time it took to get back here. Usually column `0` it's the model primary column that Backpack automatically adds when developer don't manually specify it....

Hello @tsun424 Until we merge the core support for morph to you can use `entity => false` to disable backpack behavior. Cheers

Hey @jorgetwgroup I think @maurohmartinez tackled this issue in a previous PR where we added support for focus in repeatable too. #4491

I don't think it can get easier than this. The other solution would be to use some regex in element.innerHtml and keep only the icon part or move the icon...