Skip to content

[WIP] Display All Header Content in Single Table. - #721

Open
mfixstsci wants to merge 2 commits into
spacetelescope:developfrom
mfixstsci:display-full-header
Open

[WIP] Display All Header Content in Single Table.#721
mfixstsci wants to merge 2 commits into
spacetelescope:developfrom
mfixstsci:display-full-header

Conversation

@mfixstsci

Copy link
Copy Markdown
Collaborator

Addresses #532

@pep8speaks

pep8speaks commented Apr 7, 2021

Copy link
Copy Markdown

Hello @mfixstsci, Thank you for updating !

Line 217:13: E722 do not use bare 'except'
Line 556:13: W503 line break before binary operator
Line 563:13: W503 line break before binary operator

Comment last updated at 2022-02-21 20:18:03 UTC

@mfixstsci

Copy link
Copy Markdown
Collaborator Author

@bourque if you know anyone with some more JS experience could take a look at this. It was a little buggy before my changes. The tables don't display properly when the headers are loaded.

@bourque bourque changed the title WIP Display All Header Content in Single Table. [WIP] Display All Header Content in Single Table. Oct 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants