Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [mpc-dev] Feedback on MPC 1.2 version for Kepler

Hi Christopher,

Thank you for the feedback! Comments below...

> General Issues:
> -Listings logo are a bit stretch and they don’t look sharp.

Regarding looking stretched: Do you mean they are not scaled proportionally?
That should not be the case. Could you point me to a concrete example?

Regarding sharpness: The logos are larger in 1.2 so this is more noticable
on some logos. But the problem actually existed to a varying degree prior to
1.2. Some logos actually look a bit better in the new size, some a bit worse.

I was meaning to open a bug for that after EclipseCon but hadn't gotten
around to it yet. Here it is now:

https://bugs.eclipse.org/bugs/show_bug.cgi?id=404895

I already toyed with replacing the low-quality SWT image scaling with a much
smoother one using AWT/Java2D, which gives very nice results. However, maybe
we should also discuss asking the server for a correctly scaled image and
only using client-side scaling as a fallback?

>  > - 398200: MPC Popular tab is fetching the wrong feed
>  > https://bugs.eclipse.org/bugs/show_bug.cgi?id=398200
> 
> This is working!
> 
> We need to re-instate the most favorite call to actually return the most 
> favorites : https://bugs.eclipse.org/bugs/show_bug.cgi?id=398203
> 
> I think we should fix this after Kepler is released. If we do this now, 
> MPC 1.1 will show the most favorites under the popular tab.

Sounds good to me. MPC 1.2 should be fetching the right feed already
(https://bugs.eclipse.org/bugs/show_bug.cgi?id=398200). So the server-side
transition should work smoothly once the new version has been sufficiently
disseminated.

>  > - 323259: Show "Star Rating" in MPC wizard when searching
>  > https://bugs.eclipse.org/bugs/show_bug.cgi?id=323259
> 
> Clicking on the favorite button always redirect to "Eclipse site login".
> 
> I was thinking that we could redirect the user to the actual listing and 
> let the website redirect the user to the site login. 

I did it that way to have a solution for EclipseCon, because the conditional
login scheme (see below) wasn't working and the solution pages don't prompt
you to login when you try to vote - clicking the star just does nothing...

> Maybe we could add 
> an argument/flag to the url so that I can verify if the user has a valid 
> session. If he doesn't I can redirect him to the Eclipse site login. I 
> was thinking we could do something like 
> marketplace.eclipse.org/content/jaspersoft-studio/verify_login/

That would be great.

It looks like this should do just that, but it seems to be broken (e.g. at
the bottom of a solution's page, click on "Login to flag this resource as
out of date."):

https://marketplace.eclipse.org/login/sso?redirect=content/jaspersoft-studio

If you're not logged in, it currently redirects to

https://dev.eclipse.org/site_login/?takemeback=https://marketplace.eclipse.org/login/sso?redirect=content/uml-lab

which in turn always redirects to the marketplace frontpage after login.

I've filed https://bugs.eclipse.org/bugs/show_bug.cgi?id=404898 and, to
track the MPC side, https://bugs.eclipse.org/bugs/show_bug.cgi?id=404899.

> 
>  > - 337774: No multiselect plugin install
>  > https://bugs.eclipse.org/bugs/show_bug.cgi?id=337774
> 
> +1 I was able to install 2 plugins. Very cool feature.
> 
>  >
>  > - 374105: Improve error message when there is no connectivity with the
>  > marketplace server https://bugs.eclipse.org/bugs/show_bug.cgi?id=374105
> 
> If you lose your connection while browsing MPC:
> 
> MarketplaceDiscoveryStrategy failed with an error
> Unable to read repository at 
> http://marketplace.eclipse.org/popular/top/api/p?[...]
> Connection reset

I'll look into that. I take it from the error that you opened MPC,
disconnected and switched to the popular tab?

>  > - 401721: Add a News tab to the MPC wizard
>  > https://bugs.eclipse.org/bugs/show_bug.cgi?id=401721
> 
> After clicking on the newsletter tab, I had to re-size MPC because the 
> newsletter is larger than my MPC window.
> The MPC window width seems to be relative to your screen size, a larger 
> screen might not have this problem.

The window width is determined by the preferred size of the initial page.

For the MPC, this comes from the search tab, more specifically the
marketplace switcher, which is currently hardwired to a preferred width of
500px.

To accomodate the news page's width, the wizard would have to grow
horizontally by about 25%. I experimented with changing the width in a
number of ways, but in the end I decided to leave it as-is for now for a
couple of reasons:

For one, the news page is currently loaded lazily when first switched to, so
its preferred size isn't known initially.

I experimented with making the MPC window wider by default to match the
Eclipse newsletter's width. While the solution entries do relayout
gracefully, they look a lot better at the current width.

I also experimented with changing the dialog width when switching to the
news tab, but didn't like how that made the dialog "bounce around". But I
think that's the best solution if we want to accomodate the news page
without horizontal scrolling. I can activate that in a nightly to see how it
feels.

I've added this as a comment on the bug, too. Maybe we can discuss the
details there?


Best,
Carsten
-- 
Carsten Reckord
  t  +49 561 5743277-33
  f  +49 561 5743277-8833
  e  reckord@xxxxxxxx

Yatta Solutions GmbH
  Sitz der Gesellschaft: Kassel
  Amtsgericht Kassel, HRB 14720
  USt-IdNr DE263191529

Geschäftsführung:
  Johannes Jacop,
  Dr. Christian Schneider

Adresse:
  Ludwig-Erhard-Straße 12
  34131 Kassel

Kontakt:
  t  +49 561 5743277-0
  f  +49 561 5743277-88
  e  info@xxxxxxxx

Bankverbindung:
  Kasseler Bank eG
  BLZ 520 900 00
  Kto-Nr 158 305



Back to the top