Launchpad itself

Code review comment for lp:~wallyworld/launchpad/subscription-policy-text-912159

  1. subscription-policy-text-912159
  2. Merge into devel
Revision history for this message
Curtis Hovey (sinzui) wrote :

I like the implementation (I really like the reuse of the exception). Why did you define the 'inline-informational' class? It looks like "sprite info"...sprite is inline and info is informational. Did you try that and was the layout broken?

review: Needs Information (code + ui)

« Back to merge proposal