Review of attachment 298001:
Looks good to me, otherwise. Great that the EXTERNAL policy found an actual use case!
::: src/terminal-screen-container.c @@ +107,3 @@ + GTK_POLICY_NEVER, + GTK_POLICY_ALWAYS); + gtk_container_add (GTK_CONTAINER (priv->sw), GTK_WIDGET (priv->screen));
it would be more idiomatic to say
gtk_scrolled_window_new (NULL, NULL); gtk_container_add (GTK_CONTAINER (priv->sw), GTK_WIDGET (priv->screen));
and have the two deal with their adjustments internally...
Review of attachment 298001:
Looks good to me, otherwise. Great that the EXTERNAL policy found an actual use case!
::: src/terminal- screen- container. c
@@ +107,3 @@
+ GTK_POLICY_NEVER,
+ GTK_POLICY_ALWAYS);
+ gtk_container_add (GTK_CONTAINER (priv->sw), GTK_WIDGET (priv->screen));
it would be more idiomatic to say
gtk_scrolled_ window_ new (NULL, NULL);
gtk_container_add (GTK_CONTAINER (priv->sw), GTK_WIDGET (priv->screen));
and have the two deal with their adjustments internally...