Mailing-List: contact dev-help@ofbiz.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@ofbiz.apache.org
Subject: Re: [Site] OFBIZ-9665
To: dev@ofbiz.apache.org
References: <b3538119-1233-e480-face-fe18c09154ec@les7arts.com>
	<CAFqjAi1XUDwwAJ09A_c7wnBkgC+-69mNzNnczv-iwrhSXH76WQ@mail.gmail.com>
	<CAPHhwd=hY4HUqNEmovZZG7GoUHeGA9iSQ_0-W6cRqXTR+G+aDw@mail.gmail.com>
	<CANE8kk-MTDs7YMnAaSH9XpHDjEntouzVZ1iqZH0icxg5zxBooA@mail.gmail.com>
	<4e69358e-072f-83c5-1f81-b6390516745a@les7arts.com>
	<CAPHhwdkAdj2JH=Lz8cF-tn_f+d=BkYr1gJp9v6GFywAKFpw8NQ@mail.gmail.com>
	<CAJM_vA1--fkRx5FV__xhWO2=Orn1qMkCTfv1phfEbAkCm1=bvg@mail.gmail.com>
From: Jacques Le Roux <jacques.le.roux@les7arts.com>
Organization: Les Arts Informatiques
Message-ID: <369f1205-6024-9591-1568-728f431675ee@les7arts.com>
Date: Wed, 4 Oct 2017 10:40:54 +0200
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:52.0) Gecko/20100101
	Thunderbird/52.3.0
MIME-Version: 1.0
In-Reply-To: 
	<CAJM_vA1--fkRx5FV__xhWO2=Orn1qMkCTfv1phfEbAkCm1=bvg@mail.gmail.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit
Content-Language: en-GB
archived-at: Wed, 04 Oct 2017 08:40:37 -0000

Done at r1811050

Jacques


Le 03/10/2017 à 12:07, Aditya Sharma a écrit :
> Interesting. Thanks Deepak for sharing that.
>
> +1 for having target="external" at all places as consistency is key to good
> user experience.
>
> Thanks and Regards,
>
> *Aditya Sharma* | Enterprise Software Engineer
> HotWax Systems <http://www.hotwaxsystems.com/>
> <https://www.linkedin.com/in/aditya-sharma-78291810a/>
>
> On Tue, Oct 3, 2017 at 2:00 PM, Devanshu Vyas <vyas.devanshu16@gmail.com>
> wrote:
>
>> Thanks Deepak, I looked for similar info, but was unable to find. And in
>> the meantime, got to know about the Rel attribute, so I thought the
>> intended code was rel rather than target='external'.
>> Now, with the new information at hand, I think target="_blank" or
>> target="external" both are correct. All we need to decide is whether to
>> open all links in separate tabs everytime link is clicked or just once.
>>
>> I think we should open all external links with target="external". WDYAS?
>>
>> Thanks & Regards,
>> Devanshu Vyas.
>>
>> On Tue, Oct 3, 2017 at 1:58 PM, Jacques Le Roux <
>> jacques.le.roux@les7arts.com> wrote:
>>
>>> Ha thanks Deepak,
>>>
>>> This is interesting (and surprising), so we should change ALL
>>> target="_blank" to target="external" rather for consistency sake
>>>
>>> If everybody agree I will do so
>>>
>>> Jacques
>>>
>>>
>>>
>>> Le 03/10/2017 à 10:18, Deepak Dixit a écrit :
>>>
>>>> Hi Devanshu,
>>>>
>>>> I think target="external" is correct.
>>>>
>>>> https://www.quackit.com/html/codes/html_open_link_in_new_window.cfm
>>>>
>>>> {info}
>>>> Instead of calling your window _blank, you can give it a name of your
>>>> choosing. Then all URLs will load in that same tab (as long as you use
>> the
>>>> same name).
>>>>
>>>> For example, if you have a policy of loading all external links in a
>>>> separate tab, you could call that tab, say, external. Then each link
>> that
>>>> contains target="external" will open in that window/tab. Doing this
>>>> ensures
>>>> that new windows aren't going to be popping up every time a user clicks
>> on
>>>> a link.
>>>>
>>>> {info}
>>>>
>>>> Thanks & Regards
>>>> --
>>>> Deepak Dixit
>>>> www.hotwaxsystems.com
>>>> www.hotwax.co
>>>>
>>>> On Tue, Oct 3, 2017 at 1:40 PM, Devanshu Vyas <
>> vyas.devanshu16@gmail.com>
>>>> wrote:
>>>>
>>>> Hello,
>>>>> I agree with Jacques and Aditya, we should set all external links with
>>>>> target="_blank". I have also mentioned the same on the JIRA ticket
>>>>> <https://issues.apache.org/jira/browse/OFBIZ-9665>.
>>>>> I think the intended use is rel="external", which was somehow written
>> as
>>>>> target="external", refer: https://www.w3schools.com/Tags/att_a_rel.asp
>>>>>
>>>>> Hope this helps!
>>>>>
>>>>> Thanks & Regards,
>>>>> Devanshu Vyas.
>>>>>
>>>>> On Tue, Oct 3, 2017 at 12:18 PM, Taher Alkhateeb <
>>>>> slidingfilaments@gmail.com
>>>>>
>>>>>> wrote:
>>>>>> AFAIK "_blank" is the only acceptable target in HTML 5 because frames
>>>>>> are deprecated. The website is modern and built on an HTML 5 template.
>>>>>> Hence every target should be "_blank"
>>>>>>
>>>>>> On Tue, Oct 3, 2017 at 9:42 AM, Jacques Le Roux
>>>>>> <jacques.le.roux@les7arts.com> wrote:
>>>>>>
>>>>>>> Hi,
>>>>>>>
>>>>>>> Aditya and I want to change all "external" targets to "_blank" but
>> are
>>>>>>> unsure of the reason of "external" targets
>>>>>>>
>>>>>>> Could someone please confirm it's not an issue and maybe explain why
>> we
>>>>>> have
>>>>>>
>>>>>>> "external" targets
>>>>>>>
>>>>>>> Thanks
>>>>>>>
>>>>>>> Jacques
>>>>>>>
>>>>>>>