Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

2.0 QE #2266

Closed
25 of 26 tasks
llorca opened this issue Mar 17, 2018 · 9 comments
Closed
25 of 26 tasks

2.0 QE #2266

llorca opened this issue Mar 17, 2018 · 9 comments

Comments

@llorca
Copy link
Contributor

llorca commented Mar 17, 2018

Aggregating all the issues and regressions that show up in 2.0 release candidates.

Docs

  • in "Dialogs", the example dialog body text is 16px instead of the normal 14px UI text, because of a pt-running-text above in the tree

screen shot 2018-03-19 at 8 03 10 pm

  • in "Popovers" (dark theme only), the colored text in position buttons need to be lighter
    image

  • selected low level nav item no longer updates on page scroll
    screen recording google chrome

  • Color scheme "Copied to clipboard" confirmation has wrong icon (though color palette one is fine)
    image

  • Some input shadows are cut off again
    image
    image

  • Banner seems to only show when the URL hash is declared but empty (e.g. /index.html#). in some rare cases it appears half cut off as well
    image

  • "Color aliases" table styles are broken
    image

  • pages ending with props table lack bottom padding (and have too much top margin) no repro
    image

  • remove left icon of anchor button example useful to show this case
    image

  • second alert example should be intent danger
    image

  • move alignment button group to second column
    image

  • in "Callouts" & "Radios", broken interface description (only when outside of the docs API overlay)
    image
    image

  • "Collapsible list" & "Context menus" & "Editable text" & "Labels" & "Form groups" & "Overlays" & a bunch of other pages end with almost no padding no repro
    image

  • "Dialogs" & "Checkboxes" & "Text inputs" pages end with no padding no repro
    image

  • make this gross links interactive tags instead (blocked on [Tags] Add pt-interactive support #2251)
    image

  • in "Checkboxes" & "Switches" & "Radios" & "Text input groups", .pt-align-right-style classes shouldn't wrap
    image

  • in "Navbars", there's a bunch of empty props table. either hide "Props" and "Description" headers, or hide the whole thing
    image

Core

  • file input button lost shadow (only light theme)
    image
  • numeric input buttons are no longer sized properly (both light and dark themes)
    image

Datetime

  • in "Date range picker", wrong color for month selection caret icons
    image

  • in "Date range picker", wrong alignment for reverse month/year selection
    image

  • date input shadow is gone
    image

  • in "Date range picker", mispositioned icon for month selection with disabled calendar
    image

  • time picker shouldn't have default margins/paddings
    image

  • in "Date input", "no date" label too close to input
    image

  • in "Date range input", "no date" label moves on input focus intentional so you can see the label when oppover is open
    screen recording google chrome

  • in "Date range input", focusing the input with a selected range gives broken visuals
    image

Table

  • for every table example, margin is needed between View source on GitHub and the table
    image

  • "Editing" example displays a broken table
    image

  • IRegion doesn't render the proper interface description element
    image

  • RenderMode shouldn't have a copyright header as interface description
    image

  • "Reordering" table example displays broken icon
    image
    image

@sei15
Copy link

sei15 commented Mar 20, 2018

#2137 Any chance of this getting fixed in the rcs?

@llorca
Copy link
Contributor Author

llorca commented Mar 20, 2018

@sei15 #2137 isn't a blocker to ship 2.0, so probably not as we're pretty strapped here. but we'd happily accept a contribution if you'd like to take a look at it!

@sei15
Copy link

sei15 commented Mar 20, 2018

I'll give it a shot

@giladgray
Copy link
Contributor

remaining issues: @cmslewis will tackle the table issues, i'll do the DatePicker caption ones. not gonna worry about the rest today (small docs stuff).

@cmslewis
Copy link
Contributor

cmslewis commented Mar 21, 2018

@giladgray @llorca FYI if you refresh the page while looking at the Table Features, you see this on page load:

image

I think the docs app still suffers from a Flash of Unstyled Content bug that's causing the table to mis-measure what "width: 100%" is (amongst other issues, e.g. with popover examples mis-measuring what it means to be horizontally centered in the eventual 740-px wide page).

@giladgray
Copy link
Contributor

@cmslewis that sounds in line with what i've observed. can we add an rAF somewhere to delay the measuring?

@cmslewis
Copy link
Contributor

Working on it.

@cmslewis
Copy link
Contributor

cmslewis commented Mar 21, 2018

@giladgray this is what it took. 🙄

    public componentWillMount() {
        requestAnimationFrame(() => {
            this.hasDelayedBeforeInitialRender = true;
            this.forceUpdate();
        });
    }

    public render() {
        if (!this.hasDelayedBeforeInitialRender) {
            return null;
        }
        // ...
    }

@giladgray
Copy link
Contributor

filed #2292 for follow up. we did it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants