After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 780372 - "Compact" mode for route lables in transit results aren't actually used
"Compact" mode for route lables in transit results aren't actually used
Status: RESOLVED FIXED
Product: gnome-maps
Classification: Applications
Component: general
git master
Other Linux
: Normal normal
: ---
Assigned To: gnome-maps-maint
gnome-maps-maint
Depends on:
Blocks:
 
 
Reported: 2017-03-21 20:26 UTC by Marcus Lundblad
Modified: 2017-04-03 21:36 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Screenshot showing non-compacted route labels (1.44 MB, image/png)
2017-03-21 20:27 UTC, Marcus Lundblad
  Details
transitItineraryRow: Actually use compact route labels (1.14 KB, patch)
2017-03-21 20:29 UTC, Marcus Lundblad
committed Details | Review
Screenshot after patch (1.44 MB, image/png)
2017-03-21 20:30 UTC, Marcus Lundblad
  Details

Description Marcus Lundblad 2017-03-21 20:26:42 UTC
There is two special cases when drawing the icons for individual transit legs in the transit itinerary overview (that comes up as the result of a transit route search), when there's many legs (more than 5), they are using a contracted mode where only the "mode of transportation" icon is shown. The other case, with at least three legs the route label is supposed to be drawn in a more compact manner (using some heuristics such as showing the agency name if that is shorter than the route name).
However, currently this doesn't work as intended. It has probably been lost, since one parameter is using the wrong variable.
Comment 1 Marcus Lundblad 2017-03-21 20:27:37 UTC
Created attachment 348444 [details]
Screenshot showing non-compacted route labels
Comment 2 Marcus Lundblad 2017-03-21 20:29:23 UTC
Created attachment 348445 [details] [review]
transitItineraryRow: Actually use compact route labels

Use the compact mode for route labels properly when appropriate.
Comment 3 Marcus Lundblad 2017-03-21 20:30:04 UTC
Created attachment 348446 [details]
Screenshot after patch
Comment 4 Mattias Bengtsson 2017-04-03 20:57:20 UTC
Review of attachment 348445 [details] [review]:

LGTM!
Comment 5 Marcus Lundblad 2017-04-03 21:35:51 UTC
Attachment 348445 [details] pushed as a121e92 - transitItineraryRow: Actually use compact route labels
Comment 6 Marcus Lundblad 2017-04-03 21:36:58 UTC
(In reply to Marcus Lundblad from comment #5)
> Attachment 348445 [details] pushed as a121e92 - transitItineraryRow:
> Actually use compact route labels

Thanks for the review! :)