On Mon, Jan 12, 2009 at 7:13 PM, Jakub Narebski <jnareb@xxxxxxxxx> wrote:
> On Mon, 12 Jan 2009, Giuseppe Bilotta wrote:
>> On Sun, Jan 11, 2009 at 8:15 PM, Jakub Narebski <jnareb@xxxxxxxxx> wrote:
>
>> > I see possible the following alternate solutions:
>> > * Ignore this issue (e.g. if it does not affect modern browsers)
>>
>> That would be my current choice until we find a better solution.
>
> By the way, how common this error is? Could you check if _your_ web
> browser (Firefox, Internet Explorer, Opera, Konqueror, Safari, Chrome)
> does show this bug or not, please?
Opera works fine (no display or functionality anomaly). That makes
sense, since I was the one who submitted the patch 8-D. Konqueror
3.5.9 does the ugly thing instead.
Notice that nested links are actually valid *XML*. Indeed, I asked on
www-style and they suggested leaving the problem as-is, serving as
html+xml which is what we do.
>> > * Revert 4afbaef (we lose feature, but how often used is it?)
>> > * Always use quirks mode, or check browser and use quirks mode if it
>> > would break layout
>> > * Use extra divs and links and CSS positioning to make layout which
>> > looks like current one, and behaves as current one, but is more
>> > complicated.
>>
>> I'm asking on #html, hopefully I'll get some interesting idea to try for this.
>
> I have found _a_ solution. Perhaps not the best one, but it works.
> And IMHO gives / can give even better visual.
>
> Current version (line wrapped for better visibility):
> <div class="header">
> <a class="title" href="...">GIT 1.6.1
> <span class="refs">
> <span class="tag indirect" title="tags/v1.6.1">
> <a href="...">v1.6.1</a>
> </span>
> </span>
> </a>
> </div>
>
> Current CSS (relevant part):
> a.title {
> display: block;
> padding: 6px 8px;
> }
>
> Current rendering:
> -----------------------------------------------------------
> |_GIT 1.6.1_ [] |
> -----------------------------------------------------------
> __v1.6.1__
>
>
> Proposed code (line wrapped for better visibility, with CSS embedded,
> which would change in final version of course). Only parts of style
> related to positioning are shown.
> <div class="header">
> <a href="..." style="float: left; margin: 6px 1px 6px 8px;">GIT 1.6.1</a>
> <div style="float: left; margin: 6px 1px;">
> <span class="refs">
> <span class="tag indirect" title="tags/v1.6.1">
> <a href="...">v1.6.1</a>
> </span>
> </span>
> </div>
> <a href="..." style="display: block; padding: 6px 8px;"> </a>
> </div>
>
> Rendering with proposed code:
> -----------------------------------------------------------
> _|_GIT 1.6.1_ [_v1.6.1_] |_
> -----------------------------------------------------------
>
> I guess that instead of additional DIV element, we could use DIV for
> .refs, or use "float: right" style with SPAN element. I have not
> checked if other variations works: this one does.
>
> What do you think?
The float thing was the second suggestion I was given on www-style.
Can you provide a patch I can apply to my tree for testing to see how
it comes up?
--
Giuseppe "Oblomov" Bilotta
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
