Browse Source

78782 added todo comments

Mao 4 years ago
parent
commit
003e9351a4

+ 7 - 0
packages/app/src/components/Navbar/GrowiSubNavigation.jsx

@@ -115,6 +115,10 @@ const GrowiSubNavigation = (props) => {
         ) }
         ) }
 
 
         <div className="grw-path-nav-container">
         <div className="grw-path-nav-container">
+          {/* TODO : display tags when this component is used in SearchResultContent too.
+              For that, refactor TagLabels such that it can be used while not depending on pageContainer
+              TASK: #80623 https://estoc.weseek.co.jp/redmine/issues/80623
+          */}
           { isAbleToShowTagLabel && !isCompactMode && !isTagLabelHidden && (
           { isAbleToShowTagLabel && !isCompactMode && !isTagLabelHidden && (
             <div className="grw-taglabels-container">
             <div className="grw-taglabels-container">
               <TagLabels editorMode={editorMode} />
               <TagLabels editorMode={editorMode} />
@@ -130,6 +134,9 @@ const GrowiSubNavigation = (props) => {
         <div className="d-flex flex-column align-items-end">
         <div className="d-flex flex-column align-items-end">
           <div className="d-flex">
           <div className="d-flex">
             <SubnavButtons isCompactMode={isCompactMode} />
             <SubnavButtons isCompactMode={isCompactMode} />
+            {/* TODO: refactor SubNavButtons in a way that it can be used independently from pageContainer
+              TASK : #80481 https://estoc.weseek.co.jp/redmine/issues/80481
+             */}
           </div>
           </div>
           <div className="mt-2">
           <div className="mt-2">
             {isAbleToShowPageEditorModeManager && !isSearchPageMode && (
             {isAbleToShowPageEditorModeManager && !isSearchPageMode && (

+ 1 - 1
packages/app/src/components/SearchPage/SearchResultContent.tsx

@@ -20,7 +20,7 @@ const SearchResultContent: FC<Props> = (props: Props) => {
     if (page.tags != null && page.tags.length > 0) { showTags = true }
     if (page.tags != null && page.tags.length > 0) { showTags = true }
     return (
     return (
       <div key={page._id} className="search-result-page mb-5">
       <div key={page._id} className="search-result-page mb-5">
-        <GrowiSubNavigationTypeAny pageId={page._id} path={page.path}></GrowiSubNavigationTypeAny>
+        <GrowiSubNavigationTypeAny isSearchPageMode pageId={page._id} path={page.path}></GrowiSubNavigationTypeAny>
         <RevisionRenderTypeAny
         <RevisionRenderTypeAny
           growiRenderer={growiRenderer}
           growiRenderer={growiRenderer}
           pageId={page._id}
           pageId={page._id}