-txt
diff --git a/src/zabapgit_page.prog.abap b/src/zabapgit_page.prog.abap
index 871464daf..0765dd3cf 100644
--- a/src/zabapgit_page.prog.abap
+++ b/src/zabapgit_page.prog.abap
@@ -109,6 +109,15 @@ CLASS lcl_gui_page_super IMPLEMENTATION.
ro_html->add( '' ).
+ IF abap_true = lcl_app=>user( )->is_favorite_repo( io_repo->get_key( ) ).
+ lv_icon = 'img/star' ##NO_TEXT.
+ ELSE.
+ lv_icon = 'img/star-grey' ##NO_TEXT.
+ ENDIF.
+ lv_icon = | |.
+ ro_html->add_anchor( iv_act = |{ gc_action-repo_toggle_fav }?{ io_repo->get_key( ) }|
+ iv_txt = lv_icon ).
+
IF lo_pback->exists( io_repo->get_key( ) ) = abap_true.
ro_html->add( 'BG' ).
ENDIF.
@@ -370,6 +379,7 @@ CLASS lcl_gui_page_super IMPLEMENTATION.
_add '.crossout { text-decoration: line-through !important; }'.
_add '.right { text-align:right; }'.
_add '.paddings { padding: 0.5em 0.5em; }'.
+ _add '.pad-sides { padding: 0 0.3em; }'.
" Structure div styles: header, footer, toc
_add '/* STRUCTURE DIVS, HEADER & FOOTER */'.
@@ -468,6 +478,17 @@ CLASS lcl_gui_page_super IMPLEMENTATION.
_add ' text-decoration: none;'.
_add ' display: block;'.
_add '}'.
+
+ _add '.dropdown_content td {'. " TODO refactor
+ _add ' text-align: left;'.
+ _add '}'.
+ _add '.dropdown_content td a {'. " TODO refactor
+ _add ' padding: 0em 0.2em;'.
+ _add '}'.
+ _add '.dropdown_content td.icon {'. " TODO refactor
+ _add ' padding: 0 3px 0 6px;'.
+ _add '}'.
+
_add '.dropdown_content a:hover { background-color: #f1f1f1 }'.
_add '.dropdown:hover .dropdown_content { display: block; }'.
_add '.dropdown:hover .dropbtn { color: #79a0d2; }'.
diff --git a/src/zabapgit_page_main.prog.abap b/src/zabapgit_page_main.prog.abap
index 7d2c04b0b..83623d221 100644
--- a/src/zabapgit_page_main.prog.abap
+++ b/src/zabapgit_page_main.prog.abap
@@ -26,12 +26,7 @@ CLASS lcl_gui_page_main DEFINITION FINAL INHERITING FROM lcl_gui_page_super.
retrieve_active_repo
RAISING lcx_exception,
render_toc
- IMPORTING it_list TYPE lcl_repo_srv=>ty_repo_tt
- RETURNING VALUE(ro_html) TYPE REF TO lcl_html_helper
- RAISING lcx_exception,
- render_toc_line
- IMPORTING io_toolbar TYPE REF TO lcl_html_toolbar
- iv_image_url TYPE string
+ IMPORTING it_repo_list TYPE lcl_repo_srv=>ty_repo_tt
RETURNING VALUE(ro_html) TYPE REF TO lcl_html_helper
RAISING lcx_exception,
build_main_menu
@@ -194,81 +189,98 @@ CLASS lcl_gui_page_main IMPLEMENTATION.
METHOD render_toc.
DATA: lo_pback TYPE REF TO lcl_persistence_background,
- lo_repo LIKE LINE OF it_list,
- lv_opt TYPE c LENGTH 1,
- lo_online TYPE REF TO lcl_html_toolbar,
- lo_offline TYPE REF TO lcl_html_toolbar,
+ lv_opt TYPE char1,
+ lv_key TYPE lcl_persistence_repo=>ty_repo-key,
+ lv_icon TYPE string,
+ lo_repo LIKE LINE OF it_repo_list,
+ lo_favbar TYPE REF TO lcl_html_toolbar,
+ lo_allbar TYPE REF TO lcl_html_toolbar,
+ lt_favorites TYPE lcl_persistence_user=>tt_favorites,
lv_repo_title TYPE string.
CREATE OBJECT ro_html.
- CREATE OBJECT lo_online.
- CREATE OBJECT lo_offline.
+ CREATE OBJECT lo_favbar.
+ CREATE OBJECT lo_allbar.
CREATE OBJECT lo_pback.
- IF lines( it_list ) = 0.
- RETURN.
- ENDIF.
+ lt_favorites = lcl_app=>user( )->get_favorites( ).
- LOOP AT it_list INTO lo_repo.
- IF mv_show = lo_repo->get_key( ).
+ LOOP AT it_repo_list INTO lo_repo.
+ lv_key = lo_repo->get_key( ).
+ IF lv_key = mv_show.
lv_opt = gc_html_opt-emphas.
ELSE.
CLEAR lv_opt.
ENDIF.
- IF lo_repo->is_offline( ) = abap_true.
- lo_offline->add( iv_txt = lo_repo->get_name( )
- iv_act = |{ c_actions-show }?{ lo_repo->get_key( ) }|
- iv_opt = lv_opt ).
- ELSE.
-
- lv_repo_title = lo_repo->get_name( ).
- IF lo_pback->exists( lo_repo->get_key( ) ) = abap_true.
- lv_repo_title = lv_repo_title && 'bg'. " Background marker
- ENDIF.
-
- lo_online->add( iv_txt = lv_repo_title
- iv_act = |{ c_actions-show }?{ lo_repo->get_key( ) }|
- iv_opt = lv_opt ).
-
+ lv_repo_title = lo_repo->get_name( ).
+ IF lo_pback->exists( lv_key ) = abap_true.
+ lv_repo_title = lv_repo_title && 'bg'. " Background marker
ENDIF.
+ READ TABLE lt_favorites TRANSPORTING NO FIELDS
+ WITH KEY table_line = lv_key.
+
+ IF sy-subrc = 0.
+ DELETE lt_favorites INDEX sy-tabix. " for later cleanup
+ lo_favbar->add( iv_txt = lv_repo_title
+ iv_act = |{ c_actions-show }?{ lv_key }|
+ iv_opt = lv_opt ).
+ ENDIF.
+
+ IF lo_repo->is_offline( ) = abap_true.
+ lv_icon = ' '.
+ ELSE.
+ lv_icon = ' '.
+ ENDIF.
+
+ lo_allbar->add( iv_txt = lv_repo_title
+ iv_act = |{ c_actions-show }?{ lv_key }|
+ iv_ico = lv_icon
+ iv_opt = lv_opt ).
ENDLOOP.
- ro_html->add( '' ) ##NO_TEXT.
+ " Cleanup orphan favorites (for removed repos)
+ LOOP AT lt_favorites INTO lv_key.
+ lcl_app=>user( )->toggle_favorite( lv_key ).
+ ENDLOOP.
- IF lo_online->count( ) > 0.
- ro_html->add( render_toc_line( io_toolbar = lo_online
- iv_image_url = 'img/repo_online' ) ).
+ " Render HTML
+ ro_html->add( ' ' ) ##NO_TEXT. " TODO refactor html & css
+ ro_html->add( ' ' ) ##NO_TEXT.
+ ro_html->add( ' ' ) ##NO_TEXT.
+
+**********************************************************************
+
+ ro_html->add( ' ' ).
+ ro_html->add( ' | ' ).
+
+ ro_html->add( '' ). " Maximize width
+ IF lo_favbar->count( ) > 0.
+ ro_html->add( lo_favbar->render( iv_sort = abap_true ) ).
+ ELSE.
+ ro_html->add( 'No favorites found. Please'
+ && ' click  icon repo toolbar to add'
+ && ' it as favourite. Choose a repo there →' ).
ENDIF.
+ ro_html->add( ' | ' ).
- IF lo_offline->count( ) > 0.
- ro_html->add( render_toc_line( io_toolbar = lo_offline
- iv_image_url = 'img/repo_offline' ) ).
- ENDIF.
+ ro_html->add( '' ).
+ ro_html->add( lo_allbar->render( iv_as_droplist_with_label = ' '
+ iv_sort = abap_true
+ iv_with_icons = abap_true ) ).
+ ro_html->add( ' | ' ).
+ ro_html->add( '
' ).
- ro_html->add( ' ' ).
+**********************************************************************
+
+ ro_html->add( ' ' ).
+ ro_html->add( ' ' ).
+ ro_html->add( ' ' ).
-
- ro_html->add( '' ).
- ro_html->add( | | ).
- ro_html->add( ' | ' ).
-
- ro_html->add( '' ).
- ro_html->add( io_toolbar->render( iv_sort = abap_true ) ).
- ro_html->add( ' | ' ).
-
- ro_html->add( '
|