Skip to content

Refactor samples 011-020 to app guidelines#597

Merged
oblomov-dev merged 1 commit intostandardfrom
claude/refactor-samples-011-020-MB716
Apr 8, 2026
Merged

Refactor samples 011-020 to app guidelines#597
oblomov-dev merged 1 commit intostandardfrom
claude/refactor-samples-011-020-MB716

Conversation

@oblomov-dev
Copy link
Copy Markdown
Member

  • Remove mv_/mo_ prefixes from all DATA attributes
  • Move client and helper methods from PUBLIC to PROTECTED section
  • Rename method names: set_view→view_display, z2ui5_on_→on_, render_tab_*→view_display, ui5_*→clean names
  • Replace single quotes with backticks for all string literals
  • Fix blank lines in class definitions and method bodies per coding conventions
  • Use IF/ELSEIF lifecycle chain instead of IF/RETURN + CASE patterns
  • Use inline check_on_event for ≤3 events, CASE for ≥4 events
  • Extract view_display for apps where main() would exceed 50 lines
  • Rename ty_row→ty_s_row, ty_chart→ty_s_chart, add ty_t_rows where used twice
  • Replace && concatenation with string templates where variables are embedded
  • Always use client->check_app_prev_stack() for shownavbutton

https://claude.ai/code/session_01HH52uVT6bQkXiuRChb1pz7

- Remove mv_/mo_ prefixes from all DATA attributes
- Move client and helper methods from PUBLIC to PROTECTED section
- Rename method names: set_view→view_display, z2ui5_on_*→on_*, render_tab_*→view_display, ui5_*→clean names
- Replace single quotes with backticks for all string literals
- Fix blank lines in class definitions and method bodies per coding conventions
- Use IF/ELSEIF lifecycle chain instead of IF/RETURN + CASE patterns
- Use inline check_on_event for ≤3 events, CASE for ≥4 events
- Extract view_display for apps where main() would exceed 50 lines
- Rename ty_row→ty_s_row, ty_chart→ty_s_chart, add ty_t_rows where used twice
- Replace && concatenation with string templates where variables are embedded
- Always use client->check_app_prev_stack() for shownavbutton

https://claude.ai/code/session_01HH52uVT6bQkXiuRChb1pz7
@oblomov-dev oblomov-dev merged commit 1bf5678 into standard Apr 8, 2026
6 checks passed
@oblomov-dev oblomov-dev deleted the claude/refactor-samples-011-020-MB716 branch April 8, 2026 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants