Menu

#1890 Godley Table dropdown not working

Backlog
closed
None
2critical
2025-10-27
2025-10-26
Steve Keen
No

The dropdown "wedge" isn't working in Edit mode--I'm guessing because the text edit window takes precedence over the dropdown. At a guess, reversing the precedence would fix the bug.

1 Attachments

Related

Bugs: #1890

Discussion

  • High Performance Coder

    That's because the triangle wedges are vestigial items left over from
    when the code supported the Godley popup window. Since Niel's refactor
    of the Godley popup, they're not needed.

    It is not feasible to have the canvas drive menus. The wedges should
    be removed from the edit mode tables.

    --


    Dr Russell Standish Phone 0425 253119 (mobile)
    Principal, High Performance Coders hpcoder@hpcoders.com.au
    http://www.hpcoders.com.au


     
  • Steve Keen

    Steve Keen - 2025-10-26

    So you have to type stock names in then? No assistance from Ravel in knowing what they are?

     
    • High Performance Coder

      On the canvas at any rate - obviously there is that support in the
      Godley popup.

      Note there may be a solution of adding some support in the context
      menu. A bit of logic to code in Typescript, but might be a good job to
      give Github copilot.

      On Sun, Oct 26, 2025 at 09:10:52AM -0000, Steve Keen wrote:

      So you have to type stock names in then? No assistance from Ravel in knowing
      what they are?

      ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

      [tickets:#1890] Godley Table dropdown not working

      Status: open
      Milestone: Backlog
      Created: Sun Oct 26, 2025 08:03 AM UTC by Steve Keen
      Last Updated: Sun Oct 26, 2025 08:03 AM UTC
      Owner: nobody
      Attachments:

      • EditWindowDropdownsNotWorking20251026.mp4 (10.4 MB; video/mp4)

      The dropdown "wedge" isn't working in Edit mode--I'm guessing because the text
      edit window takes precedence over the dropdown. At a guess, reversing the
      precedence would fix the bug.

      ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

      Sent from sourceforge.net because you indicated interest in https://
      sourceforge.net/p/minsky/tickets/1890/

      To unsubscribe from further messages, please visit https://sourceforge.net/auth
      /subscriptions/

      --


      Dr Russell Standish Phone 0425 253119 (mobile)
      Principal, High Performance Coders hpcoder@hpcoders.com.au
      http://www.hpcoders.com.au


       

      Related

      Bugs: #1890

  • Steve Keen

    Steve Keen - 2025-10-26

    That's a good thought. Users are used to the right-click menu of course, so adding one for when a user hovers over the stock name cell makes perfect sense. Ditto for flows.

    Here's a sample video for what I've like to make smoother still for future videos.:

    https://www.dropbox.com/scl/fi/qv8rdqgniycozgxhjq1fk/SelginError20251026ScreenOnly.mp4?rlkey=wd2nnxyzrxwvqz74qcspggiyd&dl=0

    Views on YouTube are really taking off now, which is perfect timing for the seed round.

     
  • High Performance Coder

    OK - I gave this task to Github Copilot - it didn't do too bad a job, maybe about a 90% job, which I fixed up, then promptly refined with testing.

    Only question is whether the location of "Import stock variables" in the context menu is optimal.

     
  • High Performance Coder

    • status: open --> closed
    • assigned_to: High Performance Coder
     

Anonymous
Anonymous

Add attachments
Cancel





MongoDB Logo MongoDB