irods-contrib / metalnx-web

Metalnx Web Application
https://metalnx.github.io/
BSD 3-Clause "New" or "Revised" License
36 stars 36 forks source link

[251] Fix naming issue #269

Closed connectbo closed 2 years ago

connectbo commented 2 years ago

This pr fixes this naming issue when modifying data object.

First option renders like this now(from rename button in action dropdown):

image

Second option (from collectionInfo page):

image

connectbo commented 2 years ago

@trel Seems like I don't have the permission to add you as reviewer. It looks good on my end, please review.

trel commented 2 years ago

Can we make both rename actions use the same dialog box?

And I think they should both look like the first option...

If there is a dialog for renaming a Collection... that should be consistent as well.

connectbo commented 2 years ago

Sure, that makes sense. I will poke around and make changes.

connectbo commented 2 years ago

Alright, dialog box looks like this now. Let me know if these look good:

rename option 1 (from rename button in action dropdown):

rename option 2 (from collectionInfo page):

The only difference on these dialog boxes is the inheritance section when renaming collections, option 2 doesn't have that. I suspect that it is because there is already an inheritance checkbox on collectionInfo page.

trel commented 2 years ago

'Data Object Name' instead of 'File name'

I'll play with the Inheritance setting and see how it should work.

connectbo commented 2 years ago

Ahh yes, sorry about that, missed that part. Should be correct now.

trel commented 2 years ago

I confirm you didn't touch any of the Inheritance stuff - let's leave it as is for now - even though I'm not sure why that checkbox is in that location on one of the dialogs... separate issue...

Yes, please # it and we'll get it merged - thanks.

connectbo commented 2 years ago

Done.