mcgilman commented on code in PR #8075:
URL: https://github.com/apache/nifi/pull/8075#discussion_r1409766288
##########
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-frontend/src/main/nifi/src/styles.scss:
##########
@@ -293,6 +293,15 @@ a:hover {
min-width: 760px;
}
+.full-screen-dialog {
+ position: absolute !important;
+ top: 25px;
+ bottom: 25px;
+ right: 25px;
+ left: 25px;
+ max-width: none !important;
+}
+
Review Comment:
You're correct that is not used right now. I was originally going to put the
service listing with the process group configuration. However, that is being
moved into its own view so this is currently unused. I thought about removing
it but left it in place because I suspected we'd need it for other features in
the future (docs, custom ui, etc). We are also not using all the other types of
dialogs currently.
That said, I'm happy to remove this and we can reintroduce it once it's
needed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]