airflow-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ash Berlin-Taylor (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (AIRFLOW-3067) (www_rbac) Flask flash messages are not categoried properly (result in 'incomplete' UI display)
Date Fri, 21 Sep 2018 10:22:00 GMT

     [ https://issues.apache.org/jira/browse/AIRFLOW-3067?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Ash Berlin-Taylor updated AIRFLOW-3067:
---------------------------------------
         Priority: Minor  (was: Critical)
    Fix Version/s: 1.10.1

> (www_rbac) Flask flash messages are not categoried properly (result in 'incomplete' UI
display)
> -----------------------------------------------------------------------------------------------
>
>                 Key: AIRFLOW-3067
>                 URL: https://issues.apache.org/jira/browse/AIRFLOW-3067
>             Project: Apache Airflow
>          Issue Type: Improvement
>          Components: webapp
>    Affects Versions: 1.10.0
>            Reporter: Xiaodong DENG
>            Assignee: Xiaodong DENG
>            Priority: Minor
>             Fix For: 1.10.1
>
>         Attachments: 1.png, 2.png
>
>
> The Flask flash messages are not categorized properly (ref: [http://flask.pocoo.org/docs/1.0/patterns/flashing/#flashing-with-categories).]
Then the default category would be 'message'. Flask AppBuilder would assign CSS class `alert-[category]`
to that message when displaying it (ref: [https://github.com/dpgaspar/Flask-AppBuilder/blob/master/flask_appbuilder/templates/appbuilder/flash.html]
). But 'alert-message' is not specified in *bootstrap-theme.css*. This makes the the flash
messages in the _*www_rbac*_ UI come with no background color (please check the screenshots
attached. We don't see this issue in _*www*_ UI as all flash messages without categories are
classed into 'info').
> Solution:
>  * Add proper category when we invoke *flash()*.
>  * We can't use category 'error'. In bootstrap-theme.css, we don't have `alert-error`
(we have `alert-danger` instead).



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Mime
View raw message