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

Extend build terminal in dashboard to full height of viewport #2519

Closed
humphd opened this issue Nov 24, 2021 · 12 comments · Fixed by #2556
Closed

Extend build terminal in dashboard to full height of viewport #2519

humphd opened this issue Nov 24, 2021 · 12 comments · Fixed by #2556
Assignees
Labels
area: css styling Anything related to CSS styling area: dashboard Related to Telescope's dashboard (the page that has stats) type: enhancement New feature or request

Comments

@humphd
Copy link
Contributor

humphd commented Nov 24, 2021

The build terminal (see https://dev.api.telescope.cdot.systems/v1/status/pages/build.html) is only occupying the top portion of the page. Let's have it extend to fill the available space vertically.

Screen Shot 2021-11-23 at 6 58 11 PM

@humphd humphd added type: enhancement New feature or request area: css styling Anything related to CSS styling area: dashboard Related to Telescope's dashboard (the page that has stats) labels Nov 24, 2021
@Yoda-Canada
Copy link
Contributor

@humphd I would like to work on this for release 0.4

@humphd
Copy link
Contributor Author

humphd commented Nov 24, 2021

@Yoda-Canada sounds good. NOTE: this isn't enough for 0.4, so look for some other things to pair it with.

@Yoda-Canada
Copy link
Contributor

Yoda-Canada commented Nov 24, 2021

@humphd where is the code? I find the code in the webpage, but can't find it on GitHub.
image

@Yoda-Canada
Copy link
Contributor

I only can find the id="terminal" and I try to "go to definition", but can't find anything.
image

@humphd
Copy link
Contributor Author

humphd commented Nov 24, 2021

Yes, it gets created dynamically by xterm.js at runtime. You need to stretch the #terminal parent element to fill the vertical space, and the terminal should fill it automatically.

@Yoda-Canada
Copy link
Contributor

@humphd I try to use the following code to solve this issue, but it doesn't change. Then I check the container size and the main class size doesn't fit the whole screen. I changed the max-height, but it not useful. Could you give some clues?

<div id="terminal">
      <script>
        var term = new Terminal();
        var fitAddon = new FitAddon();
        term.loadAddon(fitAddon);
        term.open(document.getElementById('terminal'));
        fitAddon.fit();
      </script>
</div>

image

@humphd
Copy link
Contributor Author

humphd commented Nov 25, 2021

Short of solving this myself, I'm not sure. You need to do some research, work with your classmates to try and dig into things, etc. You can do this.

@HyperTHD
Copy link
Contributor

HyperTHD commented Nov 26, 2021

Might be wrong on this as I'm not the best with css and styling in general but this is what you want right? I also had it align with the dashboard as well

2021-11-25.23-18-55.mp4

ll

@Yoda-Canada
Copy link
Contributor

@HyperTHD I think it is right. Could you tell me how to do it? thx

@HyperTHD
Copy link
Contributor

It's the height of the xterm-screen itself. I looked at the new status service related content and couldn't find out where it gets loaded into, but you'd basically want to increase the height of that, and add some border radius so that it ends up rounded at the bottom like the dashboard. Hope this helps @Yoda-Canada

@Andrewnt219
Copy link
Contributor

Closed with #2556

@Andrewnt219 Andrewnt219 linked a pull request Dec 10, 2021 that will close this issue
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: css styling Anything related to CSS styling area: dashboard Related to Telescope's dashboard (the page that has stats) type: enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants