Skip to content

Commit 50a1b7d

Browse files
committed
improve edge panel
1 parent c7752fe commit 50a1b7d

File tree

1 file changed

+31
-33
lines changed

1 file changed

+31
-33
lines changed

src/views/EdgePanel.tsx

Lines changed: 31 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -84,13 +84,7 @@ const EdgePanel: FC<{ isExpanded: boolean }> = ({ isExpanded }) => {
8484
});
8585
};
8686

87-
const updateEnableContributorOverlap = (checked: boolean) => {
88-
setEnableContributorOverlap(checked);
89-
setNavState({
90-
...navState,
91-
edgeCreationEnableContributorOverlap: checked,
92-
});
93-
};
87+
9488

9589
const updateEnableSharedOrganization = (checked: boolean) => {
9690
setEnableSharedOrganization(checked);
@@ -100,21 +94,9 @@ const EdgePanel: FC<{ isExpanded: boolean }> = ({ isExpanded }) => {
10094
});
10195
};
10296

103-
const updateEnableCommonStargazers = (checked: boolean) => {
104-
setEnableCommonStargazers(checked);
105-
setNavState({
106-
...navState,
107-
edgeCreationEnableCommonStargazers: checked,
108-
});
109-
};
11097

111-
const updateEnableDependencies = (checked: boolean) => {
112-
setEnableDependencies(checked);
113-
setNavState({
114-
...navState,
115-
edgeCreationEnableDependencies: checked,
116-
});
117-
};
98+
99+
118100

119101

120102

@@ -710,21 +692,22 @@ const EdgePanel: FC<{ isExpanded: boolean }> = ({ isExpanded }) => {
710692
5: "5",
711693
10: "10"
712694
}}
713-
onChange={(value) => updateTopicThreshold(value as number)}
695+
onChange={(value) => setTopicThreshold(value as number)}
696+
onAfterChange={(value) => updateTopicThreshold(value as number)}
714697
className="mt-2"
715698
/>
716699
</div>
717700
)}
718701
</div>
719702

720703
{/* Contributor Overlap */}
721-
<div className="mb-4">
704+
<div className="mb-4 opacity-50">
722705
<div className="d-flex align-items-center mb-2">
723706
<input
724707
type="checkbox"
725708
className="form-check-input me-2"
726-
checked={enableContributorOverlap}
727-
onChange={(e) => updateEnableContributorOverlap(e.target.checked)}
709+
checked={false}
710+
disabled
728711
/>
729712
<label className="form-label mb-0">Contributor Overlap</label>
730713
</div>
@@ -747,7 +730,8 @@ const EdgePanel: FC<{ isExpanded: boolean }> = ({ isExpanded }) => {
747730
10: "10",
748731
20: "20"
749732
}}
750-
onChange={(value) => updateContributorThreshold(value as number)}
733+
onChange={(value) => setContributorThreshold(value as number)}
734+
onAfterChange={(value) => updateContributorThreshold(value as number)}
751735
className="mt-2"
752736
/>
753737
</div>
@@ -771,13 +755,13 @@ const EdgePanel: FC<{ isExpanded: boolean }> = ({ isExpanded }) => {
771755
</div>
772756

773757
{/* Common Stargazers */}
774-
<div className="mb-4">
758+
<div className="mb-4 opacity-50">
775759
<div className="d-flex align-items-center mb-2">
776760
<input
777761
type="checkbox"
778762
className="form-check-input me-2"
779-
checked={enableCommonStargazers}
780-
onChange={(e) => updateEnableCommonStargazers(e.target.checked)}
763+
checked={false}
764+
disabled
781765
/>
782766
<label className="form-label mb-0">Common Stargazers</label>
783767
</div>
@@ -800,21 +784,22 @@ const EdgePanel: FC<{ isExpanded: boolean }> = ({ isExpanded }) => {
800784
50: "50",
801785
100: "100"
802786
}}
803-
onChange={(value) => updateStargazerThreshold(value as number)}
787+
onChange={(value) => setStargazerThreshold(value as number)}
788+
onAfterChange={(value) => updateStargazerThreshold(value as number)}
804789
className="mt-2"
805790
/>
806791
</div>
807792
)}
808793
</div>
809794

810795
{/* Dependencies */}
811-
<div className="mb-4">
796+
<div className="mb-4 opacity-50">
812797
<div className="d-flex align-items-center mb-2">
813798
<input
814799
type="checkbox"
815800
className="form-check-input me-2"
816-
checked={enableDependencies}
817-
onChange={(e) => updateEnableDependencies(e.target.checked)}
801+
checked={false}
802+
disabled
818803
/>
819804
<label className="form-label mb-0">Dependencies</label>
820805
</div>
@@ -831,6 +816,19 @@ const EdgePanel: FC<{ isExpanded: boolean }> = ({ isExpanded }) => {
831816
>
832817
Apply Edge Creation Rules
833818
</button>
819+
<p className="text-white-50 small text-center mt-2">
820+
Have other relationship ideas? Create a discussion topic at
821+
{' '}
822+
<a
823+
href="https://github.com/data-exp-lab/deepgit/discussions"
824+
target="_blank"
825+
rel="noreferrer noopener"
826+
className="link-light"
827+
>
828+
github.com/data-exp-lab/deepgit/discussions
829+
</a>
830+
.
831+
</p>
834832
</div>
835833
</div>
836834
</div>

0 commit comments

Comments
 (0)