Differential D10901 Diff 35905 source/blender/editors/space_spreadsheet/spreadsheet_column_values.hh
Changeset View
Changeset View
Standalone View
Standalone View
source/blender/editors/space_spreadsheet/spreadsheet_column_values.hh
- This file was added.
| /* | |||||
| * This program is free software; you can redistribute it and/or | |||||
| * modify it under the terms of the GNU General Public License | |||||
| * as published by the Free Software Foundation; either version 2 | |||||
| * of the License, or (at your option) any later version. | |||||
| * | |||||
| * This program is distributed in the hope that it will be useful, | |||||
| * but WITHOUT ANY WARRANTY; without even the implied warranty of | |||||
| * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | |||||
| * GNU General Public License for more details. | |||||
| * | |||||
| * You should have received a copy of the GNU General Public License | |||||
| * along with this program; if not, write to the Free Software Foundation, | |||||
| * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. | |||||
| */ | |||||
| #pragma once | |||||
| #include "BLI_string_ref.hh" | |||||
| #include "spreadsheet_cell_value.hh" | |||||
| namespace blender::ed::spreadsheet { | |||||
| /** | |||||
| * This represents a column in a spreadsheet. It has a name and provides a value for all the cells | |||||
| * in the column. | |||||
| */ | |||||
| class ColumnValues { | |||||
| protected: | |||||
| std::string name_; | |||||
| int size_; | |||||
| public: | |||||
| ColumnValues(std::string name, const int size) : name_(std::move(name)), size_(size) | |||||
| { | |||||
| } | |||||
| virtual ~ColumnValues() = default; | |||||
| virtual void get_value(int index, CellValue &r_cell_value) const = 0; | |||||
| StringRefNull name() const | |||||
| { | |||||
| return name_; | |||||
| } | |||||
| int size() const | |||||
| { | |||||
| return size_; | |||||
| } | |||||
| /* The default width of newly created columns, in UI units. */ | |||||
| float default_width_factor = 0.0f; | |||||
HooglyBoogly: I wouldn't call these factors. UI_UNIT_X is the factor, this is the width data, so I would just… | |||||
JacquesLuckeAuthorUnsubmitted Done Inline ActionsThing is, I noticed that UI_UNIT_X might not be good enough as a factor. Besides that, calling it default_width seems fine as well. JacquesLucke: Thing is, I noticed that `UI_UNIT_X` might not be good enough as a factor.
We might also have… | |||||
| }; | |||||
| /* Utility class for the function below. */ | |||||
| template<typename GetValueF> class LambdaColumnValues : public ColumnValues { | |||||
| private: | |||||
| GetValueF get_value_; | |||||
| public: | |||||
| LambdaColumnValues(std::string name, int size, GetValueF get_value) | |||||
| : ColumnValues(std::move(name), size), get_value_(std::move(get_value)) | |||||
| { | |||||
| } | |||||
| void get_value(int index, CellValue &r_cell_value) const final | |||||
| { | |||||
| get_value_(index, r_cell_value); | |||||
| } | |||||
| }; | |||||
| /* Utility function that simplifies creating a spreadsheet column from a lambda function. */ | |||||
| template<typename GetValueF> | |||||
| std::unique_ptr<ColumnValues> column_values_from_function(std::string name, | |||||
| int size, | |||||
| GetValueF get_value) | |||||
| { | |||||
| return std::make_unique<LambdaColumnValues<GetValueF>>( | |||||
| std::move(name), size, std::move(get_value)); | |||||
| } | |||||
| } // namespace blender::ed::spreadsheet | |||||
I wouldn't call these factors. UI_UNIT_X is the factor, this is the width data, so I would just call it default_width.