fidley / falv

FALV - Fast ALV Grid
https://abapblog.com
GNU General Public License v3.0
141 stars 52 forks source link

zcl_falv_column simpler set_texts #78

Closed TimoStahl closed 1 year ago

TimoStahl commented 1 year ago

Make the medium and large texts optional and use short texts instead.

Instead of

lo_falv->column('ACTIVE')->set_texts(
    exporting
      iv_text_s = 'Active'
      iv_text_m = 'Active'
      iv_text_l = 'Active'
  ).

you could use

lo_falv->column('Active')->set_texts('Active').
sonarcloud[bot] commented 1 year ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

fidley commented 1 year ago

Thanks! @TimoStahl