OpenNeo/impress
OpenNeo/impress
1
0
Fork 1
Code Pull requests Activity

Show the PosePicker, even if there are no standard poses labeled

Browse source 
This is because I'm gonna put alt styles in here too, and I figure it's
reasonable to just explain what's going on.
This commit is contained in:
Emi Matchu 2024-01-29 01:30:18 -08:00
parent 4fff8d88f2
commit 32f5d6d4a0
1 changed files with 15 additions and 22 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
app/javascript/wardrobe-2020/WardrobePage/PosePicker.js
View file

@ -139,15 +139,9 @@ function PosePicker({
return null;
}
// If there's only one pose anyway, don't bother showing a picker!
// (Unless we're Support, in which case we want the ability to pop it open to
// inspect and label the Unknown poses!)
const numAvailablePoses = Object.values(poseInfos).filter(
(p) => p.isAvailable,
const numStandardPoses = Object.values(poseInfos).filter(
(p) => p.isAvailable && STANDARD_POSES.includes(p.pose),
).length;
if (numAvailablePoses <= 1 && !isSupportUser) {
return null;
}
const onChange = (e) => {
dispatchToOutfit({ type: "setPose", pose: e.target.value });
@ -218,8 +212,7 @@ function PosePicker({
onChange={onChange}
initialFocusRef={initialFocusRef}
/>
{numAvailablePoses <= 1 && (
<SupportOnly>
{numStandardPoses == 0 && (
<Box
fontSize="xs"
fontStyle="italic"
@ -227,11 +220,9 @@ function PosePicker({
opacity="0.7"
marginTop="2"
>
The empty picker is hidden for most users!
<br />
You can see it because you're a Support user.
We're still working on labeling these! For now,
we're just giving you one of the poses we have.
</Box>
</SupportOnly>
)}
</>
)}
@ -371,6 +362,8 @@ const GENDER_PRESENTATION_STRINGS = {
SICK_FEM: "Feminine",
};
const STANDARD_POSES = Object.keys(EMOTION_STRINGS);
function PoseOption({
poseInfo,
onChange,