YGGverse/Yoda
1
0
Fork 0
mirror of https://github.com/YGGverse/Yoda.git synced 2025-01-28 20:14:13 +00:00
Code Issues Projects Releases Wiki Activity

fix tab items order

Browse Source
This commit is contained in:
yggverse 2024-10-08 07:09:41 +03:00
parent 0a0d7b0481
commit de09688dd8
3 changed files with 36 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
src/app/browser/window/tab.rs
View File

@ -250,22 +250,24 @@ impl Tab {
// Delegate save action to childs
let id = Database::last_insert_id(transaction);
// Read HashMap index collected
let mut page_number = 0;
// @TODO incorrect order
for (_, item) in self.index.borrow().iter() {
item.save(
transaction,
&id,
&match self.widget.gobject().current_page() {
Some(number) => number == page_number,
None => false,
},
)?;
page_number += 1;
}
// At least one active page wanted to continue
if let Some(current_page) = self.widget.gobject().current_page() {
// Read collected HashMap index
for (_, item) in self.index.borrow().iter() {
// Get page number as HashMap does not keep order, no page_reorder listener also
match self.widget.gobject().page_num(item.page()) {
Some(page_number) => {
item.save(
transaction,
&id,
&page_number,
&(current_page == page_number),
)?;
}
None => panic!(), // page number expected at this point @TODO Err?
}
}
};
}
Err(e) => return Err(e.to_string()),
}

8
src/app/browser/window/tab/item.rs
View File

@ -170,9 +170,15 @@ impl Item {
&self,
transaction: &Transaction,
app_browser_window_tab_id: &i64,
page_number: &u32,
is_initially_current: &bool,
) -> Result<(), String> {
match Database::add(transaction, app_browser_window_tab_id, is_initially_current) {
match Database::add(
transaction,
app_browser_window_tab_id,
page_number,
is_initially_current,
) {
Ok(_) => {
let id = Database::last_insert_id(transaction);

14
src/app/browser/window/tab/item/database.rs
View File

@ -17,6 +17,7 @@ impl Database {
(
`id` INTEGER PRIMARY KEY AUTOINCREMENT NOT NULL,
`app_browser_window_tab_id` INTEGER NOT NULL,
`page_number` INTEGER NOT NULL,
`is_initially_current` INTEGER NOT NULL
)",
[],
@ -26,14 +27,20 @@ impl Database {
pub fn add(
tx: &Transaction,
app_browser_window_tab_id: &i64,
page_number: &u32,
is_initially_current: &bool,
) -> Result<usize, Error> {
tx.execute(
"INSERT INTO `app_browser_window_tab_item` (
`app_browser_window_tab_id`,
`page_number`,
`is_initially_current`
) VALUES (?, ?)",
[app_browser_window_tab_id, &(*is_initially_current as i64)],
) VALUES (?, ?, ?)",
[
app_browser_window_tab_id,
&(*page_number as i64),
&(*is_initially_current as i64),
],
)
}
@ -42,7 +49,8 @@ impl Database {
"SELECT `id`,
`app_browser_window_tab_id`,
`is_initially_current` FROM `app_browser_window_tab_item`
WHERE `app_browser_window_tab_id` = ?",
WHERE `app_browser_window_tab_id` = ?
ORDER BY `page_number` ASC", // just order by, no store in struct wanted
)?;
let result = stmt.query_map([app_browser_window_tab_id], |row| {