gnezim/flights_web
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Drop duplicate leg row on direct schedule details

Browse Source 
For a single-leg flight the FlightCard summary already shows
SVO → ALA with times; the extra 'SVO→ALA / 00:05 - 06:30' line
below was redundant noise. Render the per-leg row only when the
route has multiple legs (transfer case).
This commit is contained in:
Anton
2026-04-18 13:58:17 +03:00
parent 13fb633ec4
commit 22025b3ab4
1 changed files with 21 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/features/schedule/components/ScheduleDetailsPage.tsx
+21 -18
View File
@@ -154,25 +154,28 @@ export const ScheduleDetailsPage: FC<ScheduleDetailsPageProps> = ({
<FlightCard flight={flight} />
{legs.map((leg, i) => (
<div key={`leg-${leg.index ?? i}`} className="schedule-details__leg" data-testid={`leg-${leg.index ?? i}`}>
<div className="schedule-details__leg-route">
<span>{leg.departure.scheduled.airportCode}</span>
<span>&rarr;</span>
<span>{leg.arrival.scheduled.airportCode}</span>
</div>
<div className="schedule-details__leg-times">
<span>{leg.departure.times.scheduledDeparture.localTime}</span>
<span> - </span>
<span>{leg.arrival.times.scheduledArrival.localTime}</span>
</div>
{leg.equipment.name && (
<div className="schedule-details__aircraft">
{leg.equipment.name}
{/* Per-leg aircraft info (multi-leg routes only — for a
direct flight the FlightCard summary already shows the
route and times, duplicating them here is noise). */}
{legs.length > 1 &&
legs.map((leg, i) => (
<div
key={`leg-${leg.index ?? i}`}
className="schedule-details__leg"
data-testid={`leg-${leg.index ?? i}`}
>
<div className="schedule-details__leg-route">
<span>{leg.departure.scheduled.airportCode}</span>
<span>&rarr;</span>
<span>{leg.arrival.scheduled.airportCode}</span>
</div>
)}
</div>
))}
{leg.equipment.name && (
<div className="schedule-details__aircraft">
{leg.equipment.name}
</div>
)}
</div>
))}
</div>
);
})}