Code Monkey home page Code Monkey logo

Comments (5)

PShumilov avatar PShumilov commented on August 10, 2024 1

Thanks @fidley! it works

from falv.

fidley avatar fidley commented on August 10, 2024

Hi @PShumilov,
it is the same issue like #41. I don't know why it is this way, but when I want to display grid in the popup, then the instance of grid should be created inside the PBO event of the popup.

If I could solve this problem then your issue and #41 would be solved. I need to check if there is a way for that.

Maybe you have an idea?

Cheers
Łukasz

from falv.

PShumilov avatar PShumilov commented on August 10, 2024

I tried to fix the bug different ways. In case that instance of grid should be created in PBO event I invented horrible bad solution (but it works). Maybe this could lead you to find good solvation.

I added new EXPORTING parametr eo_falv like io_falv in FM Z_FALV_DISPLAY and call it in method create for popup:

rv_falv->layout_save = 'A'.
rv_falv->variant-report = sy-cprog.
rv_falv->variant-username = sy-uname.
rv_falv->grid = cast #(  rv_falv ).

if i_popup = abap_true.
  call function 'Z_FALV_DISPLAY'
    exporting
      io_falv         = rv_falv
      iv_start_row    = 1
      iv_start_column = 1
  importing
      eo_falv         = rv_falv
endif.

After create_by_copy in PBO event:

falv->pbo( iv_dynnr = iv_dynnr ).
falv->display( iv_force_grid = abap_true ).
LEAVE TO SCREEN 0.

Then replace "old" instance with instance of grid created in PBO:

if io_falv->screen eq io_falv->c_screen_popup.
  call screen io_falv->screen starting at  iv_start_column iv_start_row
                              ending at    iv_end_column iv_end_row.
  eo_falv ?= <out>->falv.
else.
  call screen io_falv->screen.
endif.

from falv.

fidley avatar fidley commented on August 10, 2024

Perfect, I haven't thought about such way! Thanks I will try to implement similar thing inside to get rid of create_by_copy :D

I'll keep you informed.

from falv.

fidley avatar fidley commented on August 10, 2024

Please update your FALV, I've used your approach but bit pimped :)

I am creating a main GUI container in the screen and then I quit, then I pass it to grid object and now copying of objects is not necessary. :)

from falv.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.