OpenPype icon indicating copy to clipboard operation
OpenPype copied to clipboard

New Publisher: Enhancements & Bugs

Open iLLiCiTiT opened this issue 3 years ago • 7 comments

Description

List of wanted or planned enhancements in new publisher UI.

Publish widget icons

  • [x] colors of enabled and disabled buttons are not much different
  • [ ] colors of buttons should match color of background (procesisng, fail, success, ...)
  • [x] better validate icon
  • [ ] (and better refresh icon?)

image

Progress bar

  • [ ] colors do not match the palette much

image

Creator dialog

  • [ ] https://app.clickup.com/t/6658547/OP-2882

    • [x] assets and tasks don't have icons in views
    • [ ] creator plugins don't have icon and description
    • [ ] add detailed description which will be showed using help button

image

Path attribute defintion

  • [ ] https://app.clickup.com/t/6658547/OP-2881

    • [ ] add Browse button
    • [ ] somehow show allowed extensions and if folders can be passed
    • [ ] (?) validate existence of paths and visually show if is not available
    • [ ] multipath
      • [ ] does not have ability to show path or edit path
      • [ ] delete button is ugly
        • [ ] maybe add delete to the bottom of view and remove selection when clicked
        • [ ] (?) add validation dialog
      • [ ] icons are dark and not visible in the view

image

image

[cuID:OP-2729]

iLLiCiTiT avatar Feb 21 '22 18:02 iLLiCiTiT

it is possible to create an instance without a task selected.

  • [ ] https://app.clickup.com/t/6658547/OP-2880

image

task is not set so I will fix it and submit image

after submitting the status of workfileMain is still not ready and no changes to the task are successful

image

m-u-r-p-h-y avatar Feb 22 '22 15:02 m-u-r-p-h-y

Found issues:

  • [ ] missing doc for BaseCreator.collect_instances and BaseCreator.update_instances (for expected arguments)
  • [ ] Create button should have some notification/animation that was pressed and instance was created
  • [ ] Duplicate identifier message could contain urls to offending plugins
  • [ ] BaseCreator.identifier maybe should return family by default, eg you wouldnt need to fill it in when you implement BaseCreator eg. creating new Creator
  • [ ] Default version field in CreatedInstance doesn't make much sense
  • [ ] NP could show full content of of CreatedInstance in json format
  • [ ] Logs content should evaluate new lines (now it smushes everything together even if there are newline characters)
  • [ ] Logs should scroll at the end automatically
  • [ ] Old Publisher lists even new creator based on BaseCreator (in case that old and new publisher should coexist)
  • [ ] NP doesn't show paths to plugins. This is helpful to locate failing plugin when debugging/creating new ones
  • [ ] I wouldn't expect to Clean render comp to be marked green as Validator failed (I am not sure if Clean.. plugin actually got triggered as it contains debug message which is not showing, so maybe green circle is just wrong)

Screenshot 2022-03-04 152256

  • [ ] Dialog thrown by CreatorError is not inheriting styles Screenshot 2022-03-09 103633
  • [ ] logging for me is working only with print not with self.log...
  • [ ] Name in dialogs should be Variant as it is not instance name, but only a variant

kalisp avatar Mar 03 '22 11:03 kalisp

  • [x] I would change this button to say confirm. submit is a bit misleading for the artist I think image

mkolar avatar Mar 08 '22 14:03 mkolar

@iLLiCiTiT I've added a few more comments in the figma mockup https://www.figma.com/file/AkF1ZkyrVdUFt8YdkdjrBo/OpenPype?node-id=0%3A1

mkolar avatar Mar 08 '22 14:03 mkolar

Could someone either change into a checkable bullet point list so we can track status of requests or separate it into dedicated issues if they are bigger issues. It's hard to read this issue and see what the current status is with resolving it.

BigRoy avatar Apr 03 '23 22:04 BigRoy

I've updated the above bullet points of mentioned issues into checklists - please check off what can be considered fixed and what is still todo @m-u-r-p-h-y @kalisp @iLLiCiTiT @mkolar. That can help define where we are at with this issue with "general comments" and can help decide whether some of the issues might need to split into dedicated issues of there own.

BigRoy avatar Apr 19 '23 12:04 BigRoy

@iLLiCiTiT @kalisp @BigRoy Is this worth transfering to AYON and add to 🚀 Extended Publisher or is it obsolete?

dee-ynput avatar Aug 14 '24 16:08 dee-ynput