Only display domain name in the context part of the panel

RESOLVED FIXED in Firefox 40

Status

Hello (Loop)
Client
P2
normal
Rank:
15
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: RT, Assigned: standard8)

Tracking

unspecified
mozilla40
Points:
1
Bug Flags:
firefox-backlog +
in-testsuite +

Firefox Tracking Flags

(firefox40 fixed)

Details

(Whiteboard: [context])

User Story

Currently when opening a panel the full URL of the active tab gets displayed in the panel within the context area.
This is taking too much space on long URLs and this bug is a requirement to change it as follows:
- Only display domain name without the "http://" header

Example:
Before: https://www.google.es/search?q=test&ie=utf-8&oe=utf-8&gws_rd=cr&ei=qHQvVaSqGMW5OKrjgZgI
After: google.es

Attachments

(1 attachment)

Comment hidden (empty)
(Reporter)

Updated

3 years ago
Blocks: 1115341
User Story: (updated)

Updated

3 years ago
Rank: 15
Flags: firefox-backlog+
Priority: -- → P2
Whiteboard: [context]
(Assignee)

Comment 1

3 years ago
Created attachment 8597915 [details] [diff] [review]
Only display domain name in the context part of the panel.

This adjusts the display to only display the hostname.
Attachment #8597915 - Flags: review?(mdeboer)
(Assignee)

Updated

3 years ago
Assignee: nobody → standard8
Iteration: --- → 40.3 - 11 May
Points: --- → 1
Comment on attachment 8597915 [details] [diff] [review]
Only display domain name in the context part of the panel.

Review of attachment 8597915 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM!

::: browser/components/loop/content/js/panel.jsx
@@ +702,5 @@
>      render: function() {
> +      var hostname;
> +
> +      try {
> +        hostname = (new URL(this.state.url)).hostname;

nit: no need for the extra braces.
Attachment #8597915 - Flags: review?(mdeboer) → review+

Comment 3

3 years ago
https://hg.mozilla.org/integration/fx-team/rev/4084f5eca1c6
(Assignee)

Updated

3 years ago
Target Milestone: --- → mozilla40
https://hg.mozilla.org/mozilla-central/rev/4084f5eca1c6
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox40: --- → fixed
Flags: in-testsuite+
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.